-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Hey Jeff! Use this script to automatically merge all the PR's #379
Comments
I guess Jeff loves us all and he'll go through that manually and say something nice for trying 😉 |
This looks awesome! Thank you so much 👍 |
Might need some more code to do some checks such as:
Those are all I can think of off the top of my head. |
Would the number of commits matter if only one file is changed? I don't think one commit was a requirement, but I might be missing something. |
Ah, must have read it wrong. As long as one file is edited all should be good then! Other considerations might be to check if the master branch was edited in the PR, as one could infer from the instructions that it's not allowed - changes should only be merged from your own custom-named branch. Or maybe this is too strict? |
I ended up putting something together with the GitHub GraphQL API to validate the PRs. I'll give @ascendedco a shoutout in the video follow-up bc this is a helpful little script. |
@codediodeio, can you please enable the script? :) |
Hi @codediodeio ,
Pretty sure you don't want to manually merge all these PR's. So you can use the below bash script to do some of the heavy lifting. You'll need to modify it for your decryption step to be automated as well, but this should be a good starting point.
Disclaimer: The auto merging will only work if there aren't any merge conflicts, so here's hoping 🤞🏻
The text was updated successfully, but these errors were encountered: