Skip to content

Conversation

@amustaque97
Copy link
Contributor

@amustaque97 amustaque97 commented Dec 23, 2025

Closes #2197

Summary

This PR configures repository rulesets only to accept CI status checks from GitHub Actions by setting the integration_id to 15368.

Changes

  • Added GITHUB_ACTIONS_INTEGRATION_ID constant (value: 15368) with verification link
  • Updated construct_ruleset() to use the GitHub Actions integration ID for required status checks

Verification

The integration ID was verified via the public GitHub API:

curl -s https://api.github.com/repos/rust-lang/rust/commits/HEAD/check-runs | grep -A 3 '"app"'

@github-actions
Copy link

github-actions bot commented Dec 23, 2025

Dry-run check results

[WARN  sync_team] sync-team is running in dry mode, no changes will be applied.
[INFO  sync_team] synchronizing crates-io
[INFO  sync_team] synchronizing github
[INFO  sync_team] 💻 Repo Diffs:
    📝 Editing repo 'rust-lang/bors':
      Rulesets:
          Branch protection for main

@marcoieni
Copy link
Member

Thanks!
One small thing. The dry run in #2201 doesn't show the content of the diff. Can you fix it? 🙏

writeln!(result, " Checks:")?;
for check in &parameters.required_status_checks {
if let Some(integration_id) = check.integration_id {
let app_name = if integration_id == GITHUB_ACTIONS_INTEGRATION_ID {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Now it should show something like below

        Rule: Required Status Checks
          Strict Policy: false
          Checks:
            - CI (GitHub Actions, integration_id: 15368)
            - test (GitHub Actions, integration_id: 15368)

}

if !is_create && !logged {
writeln!(result, " No changes")?;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I made a change here. I do remember I need to convert the macro into a general function. I will take it up in a separate PR. In this PR, I'm adding a check for a clear description. If there are no changes, it must print.

Ruleset for main
        No changes

instead of just printing

Ruleset for main

@amustaque97 amustaque97 force-pushed the fix/2197-status-check branch from 5a89739 to ff7c6b1 Compare December 23, 2025 16:42
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.

In rulesets, set the source of the checks to GitHub Actions

2 participants