-
Notifications
You must be signed in to change notification settings - Fork 30
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
Commit .github/weekly-digest.yml
in a separate PR
#17
Comments
Agree |
Absolutely agree. I didn't know there was a push that was going to happen. Also does there need to be a PR that is pushed if you're not changing any of the default settings yet? |
Yeah. Thanks for the feedback @Robdel12. Yes, a push directly to master would have been happened (This is as per the current version. I will be updating this by weekend, and will open a PR instead). 😄 And yes, no PR is needed if you are happy with the default config. |
the docs folder says to use a |
@steveoh , the bot will work only with .github/weekly-digest.yml. |
Instead of committing directly to master branch, opening a PR to add
.github/weekly-digest.yml
appears to be a better option.The text was updated successfully, but these errors were encountered: