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

Feat: developer onboarding notification added #1052

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Mayank77maruti
Copy link

Addess issue #1043

📑 Description
This PR introduces a new GitHub Action that automatically posts a comment on a pull request once it has been merged into the main branch.

@Mayank77maruti
Copy link
Author

Hey @doberst , could you please review the pr.
I am participating in a opensource competition and it would be really great if you could review this pr.

@doberst
Copy link
Contributor

doberst commented Oct 26, 2024

@Mayank77maruti - really appreciate your contribution ... unfortunately, we tend to not use a lot of custom github actions in our development process or community engagement. Do you see another way perhaps to achieve this goal without layering in another github action?

@Mayank77maruti
Copy link
Author

Thank you for your feedback on the GitHub Action!
I can use GitHub Webhook paired with a serverless function (hosted on platforms like AWS Lambda, Vercel, or Netlify). Here’s how it would work:

Webhook Setup: I’d set up a webhook on the repository that listens for the pull_request event (specifically for merged PRs).You can set this up in the Settings of your GitHub repository under Webhooks.

Serverless Function: When the webhook is triggered, it would call a serverless function. This function would:
Fetch the author's previous contributions to determine the number of merged PRs.
Post a congratulatory message on the pull request, with wording that dynamically changes based on the author’s contribution count.

GitHub API: The function would use the GitHub API with a Personal Access Token (PAT) to comment directly on the PR, without adding any custom GitHub Action files to the repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants