summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWild Kat <wk@users.noreply.github.com>2018-06-03 20:51:00 +0200
committerGitHub <noreply@github.com>2018-06-03 20:51:00 +0200
commitff4eaaebf749e6b91ea0ef96c3eaa36776e9a4b5 (patch)
treef8cadf94cd939613d08b359f094ff0739b5be0ef
parent4bd5446f435ec6eeb077be5c6a3a7872929efc57 (diff)
bump rubocop to 0.56 and introduce PR template (#1361)
* introduce PR template in lieu of contribution guide for now * bump rubocop to 0.56 and regenerate rubocop_todo
-rw-r--r--.github/PULL_REQUEST_TEMPLATE.md12
-rw-r--r--.rubocop_todo.yml24
-rw-r--r--oxidized.gemspec2
3 files changed, 28 insertions, 10 deletions
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
new file mode 100644
index 0000000..869b4a9
--- /dev/null
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -0,0 +1,12 @@
+## Pre-Request Checklist
+<!-- Not all items apply to each PR, but a great PR addresses all applicable items. -->
+
+- [ ] Passes rubocop code analysis (try `rubocop --auto-correct`)
+- [ ] Tests added or adapted (try `rake test`)
+- [ ] Changes are reflected in the documentation
+- [ ] User-visible changes appended to [CHANGELOG.md](/CHANGELOG.md)
+
+## Description
+<!-- Describe your changes here. -->
+
+<!-- Add a text similar to "Closes issue #" if this PR relates to an existing issue. -->
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index c6a5e0d..9db8d56 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -1,6 +1,6 @@
# This configuration was generated by
# `rubocop --auto-gen-config`
-# on 2018-06-03 10:03:33 +0200 using RuboCop version 0.55.0.
+# on 2018-06-03 20:42:10 +0200 using RuboCop version 0.56.0.
# The point is for the user to remove these configuration records
# one by one as the offenses are removed from the code base.
# Note that changes in the inspected code, or installation of new
@@ -19,7 +19,7 @@ Lint/AmbiguousBlockAssociation:
- 'lib/oxidized/model/model.rb'
- 'lib/oxidized/model/nos.rb'
-# Offense count: 681
+# Offense count: 683
Lint/AmbiguousRegexpLiteral:
Enabled: false
@@ -115,7 +115,7 @@ Lint/Void:
Exclude:
- 'lib/oxidized/model/voss.rb'
-# Offense count: 63
+# Offense count: 64
Metrics/AbcSize:
Max: 84
@@ -138,7 +138,7 @@ Metrics/ClassLength:
Metrics/CyclomaticComplexity:
Max: 28
-# Offense count: 56
+# Offense count: 55
# Configuration parameters: CountComments.
Metrics/MethodLength:
Max: 76
@@ -206,6 +206,12 @@ Performance/Casecmp:
Exclude:
- 'lib/oxidized/manager.rb'
+# Offense count: 1
+# Cop supports --auto-correct.
+Performance/InefficientHashSearch:
+ Exclude:
+ - 'lib/oxidized/node.rb'
+
# Offense count: 57
# Cop supports --auto-correct.
Performance/RedundantMatch:
@@ -337,7 +343,7 @@ Style/ConditionalAssignment:
- 'lib/oxidized/hook/githubrepo.rb'
- 'lib/oxidized/model/model.rb'
-# Offense count: 150
+# Offense count: 153
Style/Documentation:
Enabled: false
@@ -412,7 +418,7 @@ Style/LineEndConcatenation:
Exclude:
- 'lib/oxidized/hook.rb'
-# Offense count: 137
+# Offense count: 141
# Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle.
# SupportedStyles: require_parentheses, require_no_parentheses, require_no_parentheses_except_multiline
@@ -503,7 +509,7 @@ Style/ParallelAssignment:
# Offense count: 4
# Cop supports --auto-correct.
-# Configuration parameters: AllowSafeAssignment.
+# Configuration parameters: AllowSafeAssignment, AllowInMultilineConditions.
Style/ParenthesesAroundCondition:
Exclude:
- 'lib/oxidized/model/ios.rb'
@@ -707,8 +713,8 @@ Style/ZeroLengthPredicate:
- 'lib/oxidized/model/ciscosmb.rb'
- 'lib/oxidized/output/git.rb'
-# Offense count: 300
+# Offense count: 302
# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
# URISchemes: http, https
Metrics/LineLength:
- Max: 283
+ Max: 193
diff --git a/oxidized.gemspec b/oxidized.gemspec
index 8a126bb..9202335 100644
--- a/oxidized.gemspec
+++ b/oxidized.gemspec
@@ -39,5 +39,5 @@ Gem::Specification.new do |s|
s.add_development_dependency 'pry', '~> 0'
s.add_development_dependency 'rails_best_practices', '~> 1.19'
s.add_development_dependency 'rake', '~> 10.0'
- s.add_development_dependency 'rubocop', '~> 0.55.0'
+ s.add_development_dependency 'rubocop', '~> 0.56.0'
end