Skip to content

Commit

Permalink
Remove qiskit-bot notification rule for rust changes (#13198) (#13201)
Browse files Browse the repository at this point in the history
We previously had a qiskit-bot notification trigger for changes to the
rust source or cargo configuration that would list myself and
`@kevinhartman` as the relevant reviewers. While this is still true, as
Qiskit is now >16% Rust (by lines of code, excluding tests) and that
ratio will only grow over time, the two of us are hardly the only people
worth calling out specifically as reviewers of rust code. Instead of
expanding the list to include the entirety of the terra-core team
individually this just removes the custom notification rule and opts to
fallback to the default terra-core entry.

(cherry picked from commit 597f2b2)

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
  • Loading branch information
mergify[bot] and mtreinish authored Sep 20, 2024
1 parent 078d5a0 commit bec0155
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions qiskit_bot.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ notifications:
- "`@t-imamichi`"
- "`@ajavadia`"
- "`@levbishop`"
".*\\.rs$|^Cargo":
- "`@mtreinish`"
- "`@kevinhartman`"
"(?!.*pulse.*)\\bvisualization\\b":
- "@enavarro51"
"^docs/":
Expand Down

0 comments on commit bec0155

Please sign in to comment.