fix: match rulesets by branch pattern

[amustaque97]

Closes #2211

Summary
Improves ruleset diffing to match by branch pattern instead of name (enabling in-place updates rather than delete/recreate)

[github-actions]

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

[marcoieni]

looks good. I did a few small changes. Testing in #2232
if the dry run works, I will squash and merge 👍

[ubiratansoares]

Side note from my side : perhaps we could have a follow-up PR to add some snapshot tests covering common scenarios around rulesets, eventually having #2211 as test case too 🙂

[marcoieni]

Ahmed, this is the diff of the dry run:

📝 Editing repo 'rust-lang/bors':
      Rulesets:
          Ruleset for main
            Target: Branch
            Source Type: Repository
            Enforcement: Active
            Include Branches: ["refs/heads/main"]
            Rule: Pull Request
              Dismiss Stale Reviews: false
              Require Code Owner Review: false
              Require Last Push Approval: false
              Required Approving Review Count: 0
              Required Review Thread Resolution: false
            Rule: Required Status Checks
              Strict Policy: false
              Checks:
                - Test (GitHub Actions, integration_id: 15368)
                - Test Docker (GitHub Actions, integration_id: 15368)
            Rule: Merge Queue
              Timeout: 360 minutes
              Grouping Strategy: Allgreen
              Merge Method: Merge
              Max Entries to Build: 5
              Max Entries to Merge: 5
              Min Entries to Merge: 0
              Min Wait Time: 0 minutes
          Ruleset for main
            Deleting ruleset

It looks very verbose. Also, is it normal that it is deleting the ruleset and recreating it?

[amustaque97]

is it normal that it is deleting the ruleset and recreating it?

@marcoieni @ubiratansoares can someone please confirm we have multiple rulesets for "main" branch? This will help me understand the reason reason for delete. Ideally it shouldn't happen 🤔