Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: add dry run workflow #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Mar 26, 2024

This should help catch issues where team gets updated in a breaking way, but sync-team is not synchronized yet.

The check happens on PRs, but also repeatedly every 4 hours, to give us a heads-up if the dry run fails. It can also be executed manually to test sync-team.

This PR requires a GITHUB_API_READ_TOKEN secret to be configured for this repository. It should contain a GitHub API token with read-only permissions for the rust-lang organization.

Copy link
Member

@rylev rylev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like CI is failing due to a clippy issue, but LGTM

@Kobzol Kobzol force-pushed the ci-dry-run branch 2 times, most recently from e0085a6 to 87a0708 Compare March 27, 2024 10:57
@Kobzol Kobzol requested a review from Mark-Simulacrum March 30, 2024 15:30
@Kobzol
Copy link
Contributor Author

Kobzol commented Dec 10, 2024

Rebased. Some sort of API token or Github App still has to be configured though. Discussed here.

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.

2 participants