Skip to content
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.

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

Make r? work in our PRs #3528

Open
RalfJung opened this issue May 1, 2024 · 8 comments
Open

Make r? work in our PRs #3528

RalfJung opened this issue May 1, 2024 · 8 comments
Labels
A-meta Not about any part of Miri per se, but about shaping the environment to make something in/with Miri C-enhancement Category: a PR with an enhancement or an issue tracking an accepted enhancement

Comments

@RalfJung
Copy link
Member

RalfJung commented May 1, 2024

Is there a way that we can get r? to work without automatic reviewer assignment?

@RalfJung RalfJung added C-enhancement Category: a PR with an enhancement or an issue tracking an accepted enhancement E-good-first-issue A good way to start contributing, mentoring is available A-meta Not about any part of Miri per se, but about shaping the environment to make something in/with Miri labels May 1, 2024
@tiif
Copy link
Contributor

tiif commented May 2, 2024

Since the documentation mentioned this:

Ghost

Using r? ghost in the initial PR top-level comment when opening a PR will disable triagebot’s auto-assignment. ghost is GitHub’s placeholder account for deleted accounts. It is used here for convenience. This is typically used for rollups or experiments where you don’t want any assignments or noise.

I wonder if a hack like below would work?

[assign.adhoc_groups]
miri = ["@username1", "@username2", "@username3"]

# Triagebot will pick one person from here to assign
[assign.owners]
# match all file with "*"
"*" = ["@ghost"]

When looking up a name, triagebot will first look at ad-hoc groups, then rust-lang teams, and if it doesn’t match either of those it assumes it is a GitHub user.

assign.adhoc_groups is to choose a random person from the miri team by using r? miri, and r? @username1 will work without any configuration.

source: triagebot pr assignment documentation

bors added a commit that referenced this issue May 3, 2024
Add rustbot claim feature

Add rustbot ``claim``, ``release-assignment`` and ``assign-user`` as mentioned in #3528.

rustbot issue assignment documentation: https://forge.rust-lang.org/triagebot/issue-assignment.html
pr trigger option documentation: https://forge.rust-lang.org/triagebot/pr-assignment.html#additional-new-pr-trigger-options
@RalfJung RalfJung changed the title Setup more rustbot features: r?, claim Make r? work in our PRs May 3, 2024
@RalfJung
Copy link
Member Author

RalfJung commented May 3, 2024

I would have hoped that just not having any owners would do it, but currently we can't use r? at all for some reason...

@tiif
Copy link
Contributor

tiif commented May 3, 2024

hmm, strange. I'd like to test if @rustbot claim would work here too.

@rustbot
Copy link
Collaborator

rustbot commented May 3, 2024

Error: Parsing assign command in comment failed: ...' claim' | error: expected end of command at >| ' would wor'...

Please file an issue on GitHub at triagebot if there's a problem with this bot, or reach out on #t-infra on Zulip.

@tiif
Copy link
Contributor

tiif commented May 3, 2024

Uh oh... @rustbot claim

@tiif
Copy link
Contributor

tiif commented May 3, 2024

Nice @rustbot release-assignment

@rustbot rustbot unassigned tiif May 3, 2024
@RalfJung
Copy link
Member Author

RalfJung commented May 3, 2024

I think this is blocked on triagebot improvements:

@RalfJung
Copy link
Member Author

RalfJung commented May 3, 2024

For now apparently we can use @rustbot assign $name instead of r? and that should work in PRs as well.

@RalfJung RalfJung removed the E-good-first-issue A good way to start contributing, mentoring is available label May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-meta Not about any part of Miri per se, but about shaping the environment to make something in/with Miri C-enhancement Category: a PR with an enhancement or an issue tracking an accepted enhancement
Projects
None yet
Development

No branches or pull requests

3 participants