automate badge distribution based on merged pr activity #434
Conversation
|
Hey @phyleria , please ignore that commits of article PR, I checked out to this new branch from that article branch itself, thats why those files are also being shown as modified/added , once you merge that article PR first, it will be corrected. Thank you. |
|
Hey @phyleria , since the PR is approved, I suggest before merging this - please merge that article PR (I'll wait for your changes there) first and then do a "Update branch" here on this PR and then merge it - otherwise the flow will break. |
|
Ok, noted @adarsh-jha-dev |
@phyleria, It'd be nice if you please put the |
|
Done ✅ |
Thanks , so nice of you. |
adarsh-jha-dev
force-pushed
the
390-Automate-Badge-Distribution-Based-on-Merged-PR-Activity
branch
from
061791c to
c401ddc
Compare
|
hey @phyleria , I was just going through my PRs and saw the conflicts (as some of the PRs were merged) as I updated them, the approval got removed- could you please ask @todak2000 to approve it again? (the conflicts where not from any code files - that was just the articles which were added by me). Thank you! |
|
Hi @todak2000, Please approve @adarsh-jha-dev's PR again. Thanks! |
|
@phyleria, PR merged. Please send a chimoney.io reward to @adarsh-jha-dev |
Closes #390
To make the github action work, follow these:
Add the following secrets in Secrets and Variables on github:
-
EMAIL_USER: The email address to send badge notifications from-
EMAIL_PASS: The password or app-specific password for the email account