securesauce / precaution-beta

Precaution provides a simple, automated code review for GitHub projects by running code linters with a security focus on pull requests.
Other
1 stars 0 forks source link

Add configuration file support and exclude option #213

Closed MVrachev closed 5 years ago

MVrachev commented 5 years ago

Our users need a way to configure Precaution for their needs.

The first important option to enable will be the "exclude" option. That way the users can exclude their tests folders and files.

We decided to give globs as a way for the users to be more flexible when they are creating exclude rules.

Because YAML allows you to use comments and it's more human readable than JSON we decided to use .precaution.yaml as a config file.

Closes: https://github.com/vmware/precaution/issues/179

Signed-off-by: Martin Vrachev mvrachev@vmware.com

joshuagl commented 5 years ago

Trying to prompt Travis with a close/reopen

joshuagl commented 5 years ago

Go Travis, go!

joshuagl commented 5 years ago

@browne want to take a last look at this PR before it merges?

Browne commented 5 years ago

@Browne want to take a last look at this PR before it merges?

@joshuagl - Definitely tagged the wrong person here 😺

joshuagl commented 5 years ago

My mistake, apologies for the inconvenience @Browne

joshuagl commented 5 years ago

Giving @ericwb a chance for another look before this merges.