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

fix: autocomment on pr workflow issue #65

Merged

Conversation

ARYPROGRAMMER
Copy link
Contributor

Summary

This PR improves the current autocomment on pr workflow.

Comment on Pull Requests:
Added a detailed acknowledgment comment for PRs using peter-evans/create-or-update-comment@v3.

Add Labels:
Integrated actions-ecosystem/action-add-labels@v1 to apply the Under Review label to new pull requests.

Request Reviews:
Used actions/github-script@v6 to request reviews from @yashksaini-coder on pull requests.

Comment on Issues:
Added detailed acknowledgment comments for new issues.

Permissions:
Ensured required write permissions (issues and pull-requests) for actions.

Conditional Execution:
Configured steps to trigger only for relevant events (pull_request or issues).
Error Fix:

Resolved issues with commenting on PRs and ensured workflows execute without errors.

Fixes #40

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Other (please describe):

Testing

Can be done when merged

Screenshots/Videos

None

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Signed-off-by: Arya Pratap Singh <[email protected]>
Copy link

vercel bot commented Jan 10, 2025

@ARYPROGRAMMER is attempting to deploy a commit to the Yash Kumar Saini's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

Welcome, new contributor!

   Please make sure you've read our [Contributing Guide](README.md) and we look forward to reviewing your Pull request shortly ✨

@ARYPROGRAMMER
Copy link
Contributor Author

@yashksaini-coder please review and merge.

@yashksaini-coder yashksaini-coder added SWOC officially part of SWOC season 5 Beginner SWOC Level 1 issue labels Jan 11, 2025
Copy link
Owner

@yashksaini-coder yashksaini-coder left a comment

Choose a reason for hiding this comment

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

Thank you for contributing, but next time, make sure to first be assigned on the issue. Don't raise the PR directly. @ARYPROGRAMMER

@yashksaini-coder yashksaini-coder merged commit 982c9d2 into yashksaini-coder:main Jan 11, 2025
2 of 3 checks passed
@yashksaini-coder yashksaini-coder added the Points Assigned The points have been assigned on OS Lead label Jan 11, 2025
@ARYPROGRAMMER
Copy link
Contributor Author

Thank you for contributing, but next time, make sure to first be assigned on the issue. Don't raise the PR directly. @ARYPROGRAMMER

surely @yashksaini-coder , I forgot to do that, since it was already unassigned i went ahead to directly implement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Beginner SWOC Level 1 issue Points Assigned The points have been assigned on OS Lead SWOC officially part of SWOC season 5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Comment on PR
2 participants