ackama / rails-template

Application template for Rails 7 projects; preloaded with best practices for TDD, security, deployment, and developer productivity.
Other
294 stars 15 forks source link

fix: omit default brakeman flags #521

Closed G-Rath closed 8 months ago

G-Rath commented 9 months ago

These are the default since v4.0.0.

The actual motivator for this is that it brings the line under 80 characters so Prettier is happier, but also really we should be using config/brakeman.yml for these settings to ensure all brakeman calls are consistent by default (I've not added that though because I have not seen evidence that brakeman is in the habit of changing its defaults often, so I don't think we need to worry)