weekly-digest icon indicating copy to clipboard operation
weekly-digest copied to clipboard

Published 20 hours ago • verified publisher icon abhijeetps

Commit `.github/weekly-digest.yml` in a separate PR

Open abhijeetps opened this issue 7 years ago • 5 comments

Instead of committing directly to master branch, opening a PR to add .github/weekly-digest.yml appears to be a better option.

abhijeetps avatar Jun 13 '18 17:06 abhijeetps

Agree

gr2m avatar Jun 14 '18 01:06 gr2m

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?

Robdel12 avatar Jun 14 '18 19:06 Robdel12

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 app automatically has some default configurations, which it will use if it doesn't find .github/weekly-digest.yml file in your repository.

abhijeetps avatar Jun 15 '18 05:06 abhijeetps

the docs folder says to use a config.yaml file as well. might want to update that.

steveoh avatar Aug 30 '18 16:08 steveoh

@steveoh , the bot will work only with .github/weekly-digest.yml. Thank you for pointing that out! I have updated the docs! 😄

abhijeetps avatar Aug 30 '18 16:08 abhijeetps