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

nonspec: Suggest people assign reviewers to PRs #1154

Merged
merged 6 commits into from
Sep 25, 2024

Conversation

TomHennen
Copy link
Contributor

Suggest PR authors assign specific reviewers and ping the Slack
channel if they want.

The goal is to make it more clear who is expected to take the next
action on PRs vs the current situation where the maintainers may
think someone else will take a look or that they may have missed
the initial PR request.

fixes #1149

Suggest PR authors assign specific reviewers and ping the Slack
channel if they want.

The goal is to make it more clear who is expected to take the next
action on PRs vs the current situation where the maintainers may
think someone else will take a look or that they may have missed
the initial PR request.

fixes slsa-framework#1149

Signed-off-by: Tom Hennen <tomhennen@google.com>
Signed-off-by: Tom Hennen <tomhennen@google.com>
Signed-off-by: Tom Hennen <tomhennen@google.com>
Copy link

netlify bot commented Sep 24, 2024

Deploy Preview for slsa canceled.

Name Link
🔨 Latest commit c8dda83
🔍 Latest deploy log https://app.netlify.com/sites/slsa/deploys/66f2f368d20d2b00087b3795

Copy link
Contributor

@marcelamelara marcelamelara left a comment

Choose a reason for hiding this comment

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

Thanks @TomHennen !

CONTRIBUTING.md Outdated Show resolved Hide resolved
TomHennen and others added 3 commits September 24, 2024 13:09
Co-authored-by: Marcela Melara <marcela.melara@intel.com>
Signed-off-by: Tom Hennen <TomHennen@users.noreply.github.com>
Signed-off-by: Tom Hennen <tomhennen@google.com>
Signed-off-by: Tom Hennen <tomhennen@google.com>
Copy link
Member

@lehors lehors left a comment

Choose a reason for hiding this comment

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

LGTM

@marcelamelara marcelamelara merged commit e9e35a4 into slsa-framework:main Sep 25, 2024
6 checks passed
zachariahcox pushed a commit to zachariahcox/slsa that referenced this pull request Oct 1, 2024
Suggest PR authors assign specific reviewers and ping the Slack
channel if they want.

The goal is to make it more clear who is expected to take the next
action on PRs vs the current situation where the maintainers may
think someone else will take a look or that they may have missed
the initial PR request.

fixes slsa-framework#1149

---------

Signed-off-by: Tom Hennen <tomhennen@google.com>
Signed-off-by: Tom Hennen <TomHennen@users.noreply.github.com>
Co-authored-by: Marcela Melara <marcela.melara@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Assign individuals to review PRs
3 participants