Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - fix(.github/workflows): try a fix for review bodies#18455

Closed
alexjbest wants to merge 2 commits intomasterfrom
alexjbest/maintainer-merge-comment
Closed

[Merged by Bors] - fix(.github/workflows): try a fix for review bodies#18455
alexjbest wants to merge 2 commits intomasterfrom
alexjbest/maintainer-merge-comment

Conversation

@alexjbest
Copy link
Copy Markdown
Member

@alexjbest alexjbest commented Feb 16, 2023

cf. #18400 (review)

as reviews can only be posted on pr's we don't need these checks, and they were stopping the actions from firing.


Open in Gitpod

Copy link
Copy Markdown
Member Author

@alexjbest alexjbest left a comment

Choose a reason for hiding this comment

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

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by alexjbest.

@fpvandoorn fpvandoorn changed the title fix(.gtihub/workflows): try a fix for review bodies fix(.github/workflows): try a fix for review bodies Feb 16, 2023
@alexjbest alexjbest added the awaiting-review The author would like community review of the PR label Feb 16, 2023
ping_zulip:
name: Ping maintainers on Zulip
if: (github.event.issue.pull_request != 'null') && (startsWith(github.event.review.body, 'maintainer merge') || contains(toJSON(github.event.review.body), '\r\nmaintainer merge'))
if: (startsWith(github.event.review.body, 'maintainer merge') || contains(toJSON(github.event.review.body), '\r\nmaintainer merge'))
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by alexjbest.

@eric-wieser
Copy link
Copy Markdown
Member

bors d+

Feel free to merge if you're done with this.

Thanks!

@bors
Copy link
Copy Markdown

bors bot commented Feb 16, 2023

✌️ alexjbest can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Feb 16, 2023
@alexjbest
Copy link
Copy Markdown
Member Author

bors r+

bors bot pushed a commit that referenced this pull request Feb 16, 2023
cf. #18400 (review)

as reviews can only be posted on pr's we don't need these checks, and they were stopping the actions from firing.
@bors
Copy link
Copy Markdown

bors bot commented Feb 17, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title fix(.github/workflows): try a fix for review bodies [Merged by Bors] - fix(.github/workflows): try a fix for review bodies Feb 17, 2023
@bors bors bot closed this Feb 17, 2023
@bors bors bot deleted the alexjbest/maintainer-merge-comment branch February 17, 2023 00:41
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

delegated The PR author may merge after reviewing final suggestions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants