Skip to content

Add conclusion, completed_at and output fields to create check api#467

Merged
XAMPPRocky merged 7 commits into
XAMPPRocky:mainfrom
ikkerens:feature/add-fields-to-create-check-api
Jan 9, 2024
Merged

Add conclusion, completed_at and output fields to create check api#467
XAMPPRocky merged 7 commits into
XAMPPRocky:mainfrom
ikkerens:feature/add-fields-to-create-check-api

Conversation

@ikkerens
Copy link
Copy Markdown
Contributor

@ikkerens ikkerens commented Oct 9, 2023

The create checks api also allows for immediate reporting of its completion status for quick status reports (for example, verifying that a PR name matches a regex).

This PR adds those fields to the create check api and builder, skipping the need for another request to update the check immediately after creating it.

Comment thread src/api/checks.rs Outdated
Comment thread examples/github_app_authentication.rs
Comment thread src/api/checks.rs Outdated
@ikkerens
Copy link
Copy Markdown
Contributor Author

I've ran the formatter to fix some of the issues, though my clippy locally still fails due to code that is not mine.
I take it the github action here is running an older version of rust that does not have the checks present in rust clippy 1.73.0?

@maflcko
Copy link
Copy Markdown
Contributor

maflcko commented Nov 29, 2023

lgtm

@maflcko
Copy link
Copy Markdown
Contributor

maflcko commented Jan 9, 2024

Friendly ping @XAMPPRocky :)

@XAMPPRocky
Copy link
Copy Markdown
Owner

Thank you for your PR, and congrats on your first contribution! 🎉

@XAMPPRocky XAMPPRocky merged commit 6b477da into XAMPPRocky:main Jan 9, 2024
@github-actions github-actions Bot mentioned this pull request Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants