-
-
Notifications
You must be signed in to change notification settings - Fork 239
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
Automate comment of every PR merge. #1033 #1055
Conversation
@nikhil25803 sir, can you review this? |
@nikhil25803 sir , Can you review this? |
@nikhil25803 sir, please review it and merge it. |
Hey @Anshika14528, test this out it one of your repo and upload a screenshot/video here to validate if it is working. |
@nikhil25803 sir , i have tested it on my repo please review it |
|
||
on: | ||
pull_request_target: | ||
types: [closed] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The type should not be closed
ideally, I want the message to be sent only if it is merged successfully.
Great work @Anshika14528 !!
|
Description
As soon as a PR merge, a bot will add this message
Great work @{username} !!
Do not forget to
Resolves: [Issue Number]#1003
Checklist
Screenshots
Additional Notes/Comments
I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.