Skip to content

chore(ci): Update PR Generate Hello message#15037

Merged
voisey merged 1 commit intomagma:masterfrom
voisey:update-pr-bot
Feb 24, 2023
Merged

chore(ci): Update PR Generate Hello message#15037
voisey merged 1 commit intomagma:masterfrom
voisey:update-pr-bot

Conversation

@voisey
Copy link
Copy Markdown
Contributor

@voisey voisey commented Feb 21, 2023

Summary

Closes #14967.

Tidies up the PR bot message by:

  • Updating the title of the workflow that checks PR titles
  • Removing "enforced by" from the links to the two workflows
  • Changing one instance of "pull request" to "PR", which is used elsewhere in the message
  • Replacing the link to the outdated GitHub-to-Slack mapping with a more up-to-date section on guidelines

I also propose making the following modification to the opening paragraph of Code Review Process:

"A personal message on Slack should only be the last resort, if there is no response on a pull request after a weeks time."
->
"A personal message on Slack should only be the last resort, if there is no response on a pull request after a week's time. In such cases, the GitHub-to-Slack mapping may be useful."

as well as opening a follow-up issue to update the aforementioned list.

Test Plan

N/A

@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines. label Feb 21, 2023
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for opening a PR! 💯

A couple initial guidelines

Howto

  • Reviews. The "Reviewers" listed for this PR are the Magma maintainers who will shepherd it.
  • Checks. All required CI checks must pass before merge.
  • Merge. Once approved and passing CI checks, use the ready2merge label to indicate the maintainers can merge your PR.

More info

Please take a moment to read through the Magma project's

If this is your first Magma PR, also consider reading

@github-actions github-actions bot added the component: ci All updates on CI (Jenkins/CircleCi/Github Action) label Feb 21, 2023
@github-actions
Copy link
Copy Markdown
Contributor

DP Lint & Test

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit f303f03.

@voisey voisey requested a review from LKreutzer February 21, 2023 10:04
@voisey voisey marked this pull request as ready for review February 21, 2023 10:04
@voisey voisey requested a review from a team as a code owner February 21, 2023 10:04
@voisey voisey self-assigned this Feb 21, 2023
Copy link
Copy Markdown
Contributor

@LKreutzer LKreutzer left a comment

Choose a reason for hiding this comment

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

Unfortunately the magma teams/approver groups do not seem to be publicly visible - I don't know if updating the approver list in the Wiki is a sustainable solution?

@voisey voisey merged commit 478ff46 into magma:master Feb 24, 2023
Gmerold added a commit to Gmerold/magma that referenced this pull request Feb 24, 2023
lucasgonze pushed a commit to lucasgonze/magma that referenced this pull request Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: ci All updates on CI (Jenkins/CircleCi/Github Action) size/XS Denotes a PR that changes 0-9 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update information relevant to the PR Generate Hello workflow

2 participants