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

Added the feature to auto comment on pr merged #69

Merged
merged 2 commits into from
Jun 24, 2024
Merged

Added the feature to auto comment on pr merged #69

merged 2 commits into from
Jun 24, 2024

Conversation

suhanipaliwal
Copy link
Contributor

Related Issue

Fixes #57

Description

I have added auto-comment-on-pr-merge.yml in .github/workflows directory. This file will add the feature of auto commenting when a pr is merged.

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Checklist:

  • I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

@Yeasir0032
Copy link
Owner

You can have these messages too

Thank you for your valuable contribution! Your changes are now part of the main branch and will be included in the next release.
If you haven't done so already, consider starring our repository! Starring helps to bookmark the repository for easy access and shows your support and appreciation.

@suhanipaliwal
Copy link
Contributor Author

You can have these messages too

Thank you for your valuable contribution! Your changes are now part of the main branch and will be included in the next release. If you haven't done so already, consider starring our repository! Starring helps to bookmark the repository for easy access and shows your support and appreciation.

Added these messages as well.

@Yeasir0032
Copy link
Owner

How about you edit the file instead of creating a new one

@suhanipaliwal
Copy link
Contributor Author

How about you edit the file instead of creating a new one

I have not created a new file. I just edited the previous file.

@Yeasir0032
Copy link
Owner

Good work.
Your PR will be merged shortly.

Why you have created duplicate PR in pr number 78

@suhanipaliwal
Copy link
Contributor Author

Good work. Your PR will be merged shortly.

Why you have created duplicate PR in pr number 78

That is not a duplicate pr. It has a different purpose. Pr 78 is adding the feature of auto commenting on a new pull request being made. But this pr is adding the feature of auto commenting after a pull request is approved and merged by the admins.

@sanjay-kv sanjay-kv removed the level1 label Jun 16, 2024
@Yeasir0032 Yeasir0032 merged commit 0eee28d into Yeasir0032:main Jun 24, 2024
1 check passed
Yeasir0032 added a commit that referenced this pull request Jun 24, 2024
Yeasir0032 added a commit that referenced this pull request Jun 24, 2024
@suhanipaliwal
Copy link
Contributor Author

Could you please add the label of level1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers gssoc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto Commenting when a PR is merged
4 participants