saltstack-formulas / template-formula

SaltStack formula template filled with dummy content
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
119 stars 85 forks source link

chore(rubocop): move `LineLength` cop from `Metrics` to `Layout` #188

Closed dafyddj closed 4 years ago

dafyddj commented 4 years ago

PR progress checklist (to be filled in by reviewers)


What type of PR is this?

Primary type

Secondary type

Does this PR introduce a BREAKING CHANGE?

No.

Related issues and/or pull requests

rubocop-hq/rubocop#7542

Describe the changes you're proposing

Upstream RuboCop have moved the LineLength cop from the Metrics department to the Layout department. Although the current setting still works, running RuboCop results in a warning, and we must assume that the Metrics/LineLength setting will be deprecated at some point.

Pillar / config required to test the proposed changes

Debug log showing how the proposed changes work

Get rid of this warning message: .rubocop.yml: Metrics/LineLength has the wrong namespace - should be Layout

Documentation checklist

Testing checklist

Additional context

myii commented 4 years ago

Thanks for resolving this, @dafyddj. The change has been propagated throughout the formulas using https://github.com/myii/ssf-formula/pull/129. Appreciate the review and merge, @baby-gnu.

saltstack-formulas-travis commented 4 years ago

:tada: This PR is included in version 4.0.4 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: