Skip to content

feat: implement ruleset bypass actor #2199

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.

Sign up for GitHub

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

Jump to bottom
Open
amustaque97 wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat: implement ruleset bypass actor #2199

amustaque97 wants to merge 5 commits into from

Conversation

@amustaque97
Copy link
Contributor

@amustaque97 amustaque97 commented Dec 22, 2025

Closes: #2192

@github-actions
Copy link

github-actions bot commented Dec 22, 2025 •
edited
Loading

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:
          Ruleset for main
            No changes

@amustaque97 amustaque97 force-pushed the feat/2192-implement-ruleset-bypass-actors branch 2 times, most recently from 29a954f to 023b6b5 Compare December 22, 2025 18:45
amustaque97
amustaque97 commented Dec 23, 2025
View reviewed changes
sync-team/src/github/mod.rs
source_type: RulesetSourceType::Repository,
enforcement: RulesetEnforcement::Active,
bypass_actors: None,
bypass_actors: None, // Will be populated when applying the diff
Copy link
Contributor Author

@amustaque97 amustaque97 Dec 23, 2025

Choose a reason for hiding this comment

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

We can discuss if this needs to be changed. I can think of making ID resolution methods available on the read client.

I'm okay with this comment because the current implementation is read-only

amustaque97
amustaque97 commented Dec 23, 2025
View reviewed changes
@amustaque97 amustaque97 marked this pull request as ready for review December 23, 2025 08:01
@amustaque97 amustaque97 force-pushed the feat/2192-implement-ruleset-bypass-actors branch from e1c1ba6 to 0330575 Compare January 7, 2026 04:32
marcoieni
marcoieni reviewed Jan 13, 2026
View reviewed changes
Copy link
Member

@marcoieni marcoieni left a comment

Choose a reason for hiding this comment

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

Can you edit a toml file that can take advantage of this so that we can test this PR later? (I still need to review it before testing it)

sync-team/src/github/api/mod.rs
#[serde(rename_all = "snake_case")]
pub(crate) enum RulesetBypassMode {
Always,
#[serde(rename = "pull_request")]
Copy link
Member

@marcoieni marcoieni Jan 13, 2026

Choose a reason for hiding this comment

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

isn't rename_all = "snake_case" already doing this rename?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcoieni marcoieni marcoieni left review comments

@Mark-Simulacrum Mark-Simulacrum Awaiting requested review from Mark-Simulacrum Mark-Simulacrum is a code owner

@pietroalbini pietroalbini Awaiting requested review from pietroalbini pietroalbini is a code owner

@jdno jdno Awaiting requested review from jdno jdno is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Implement bypass actors for rulesets based on allowed_merge_teams and merge_bots

2 participants

@amustaque97 @marcoieni