issues
search
newrelic
/
nri-nagios
New Relic Infrastructure Nagios Integration
MIT License
3
stars
13
forks
source link
Relax permissions
#22
Closed
camdencheek
closed
4 years ago
camdencheek
commented
4 years ago
Description of the changes
Relax required permissions from 0600 to 0640
PR Review Checklist
Author
[x] the PR should focus on a single subject. Change only relevant files to the problem you’re working on Clean and format the code
[x] add unit tests for your changes and make sure all unit tests are passing
[x] add a risk label after carefully considering the "blast radius" of your changes
[x] describe the
intent
of your changes in the description. don't just rewrite your code in prose
[x] assign at least one reviewer
[x] address the feedback
Reviewer
[ ] review code for readability
[ ] verify that high risk behavior changes are well tested
[ ] check license for any new external dependency
[ ] ask questions about anything that isn't clear and obvious
[ ] approve the PR when you consider it's good to merge
Description of the changes
Relax required permissions from 0600 to 0640
PR Review Checklist
Author
Reviewer