Skip to content

delete disable-rulesets-repos config field#2433

Merged
marcoieni merged 1 commit intomainfrom
delete-disable-rulesets-repos-config-field
Apr 21, 2026
Merged

delete disable-rulesets-repos config field#2433
marcoieni merged 1 commit intomainfrom
delete-disable-rulesets-repos-config-field

Conversation

@marcoieni
Copy link
Copy Markdown
Member

@marcoieni marcoieni commented Apr 21, 2026

We don't need branch protections anymore since all repositories use rulesets now

@github-actions
Copy link
Copy Markdown

Dry-run check results

[WARN  rust_team::sync] sync-team is running in dry mode, no changes will be applied.
[INFO  rust_team::sync] synchronizing crates-io
[INFO  rust_team::sync] synchronizing github

@marcoieni marcoieni force-pushed the delete-disable-rulesets-repos-config-field branch from 351d3a6 to adbf39f Compare April 21, 2026 12:06
@marcoieni marcoieni force-pushed the delete-disable-rulesets-repos-config-field branch from adbf39f to 6a82ffb Compare April 21, 2026 12:13
Comment thread src/sync/github/mod.rs
fn should_use_rulesets(&self, _repo: &rust_team_data::v1::Repo) -> bool {
// TODO delete this function in the future since we don't need it anymore
true
}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I kept this function to minimize the size of this PR

@marcoieni marcoieni added this pull request to the merge queue Apr 21, 2026
Merged via the queue into main with commit 7762c95 Apr 21, 2026
4 checks passed
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