lifen-ruby-style 1.0.3 → 1.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 65e1b988bd876540a6d7885a18606a0ec7ae318b14bf0ef4da37588eff5ae93b
-  data.tar.gz: c260e574dbdd3c5da8751e909e743843275f9fe70e661e4f45c5423cfce7f3ff
+  metadata.gz: e1f292884cf4a18086997f978e16a567ff9b4c092dd29b010296e5dfb0e72b5e
+  data.tar.gz: 682b8f45833ca1c66d651cd615cb4a554f47b62f68dd2092686fb9cb1da7a081
 SHA512:
-  metadata.gz: 02a1fa372eeb95537d4b0c7ef6a36de04c4cc2973cf01db3012ba0cd492731e54b4207c449e71bd2682532c5372081b2781bc7d303dda390c2ff3c91d48466e4
-  data.tar.gz: 8dd7829b59784d538e6483e84b4b7a20fb743291caeb1c2781b00641d106958a43d4d8ee5957f5308dfb418184f737dfdbbe623a5f820c3add952c8988fb9aeb
+  metadata.gz: 2cbd1799ccf7e2a4d7e45bc60761c4287dcef0ecd25e7933cbc2d8165e6f9c8c630faa38c06a831e73bb39d2c412fcccae324cc38667428fdd5f6a86aa5ec0e0
+  data.tar.gz: 43879605b3abc391a3599bed6ac120e22b3c0401890ff10d48e150fa7a8eef7dd249d1f4d1a1f321a8d4fc9e1da1d08b91e6ef12c5449f15bc814e65d1b55e38

data/.overcommit.yml

@@ -19,6 +19,7 @@ PreCommit:
   RuboCop:
     enabled: true
     command: ["bundle", "exec", "rubocop"]
+    problem_on_unmodified_line: ignore
 
   GoFmt:
     enabled: false

data/README.md

@@ -20,6 +20,7 @@
   - [Overcommit](#commandes-overcommit)
 - [Utilisation d'une version locale](#utilisation-dune-version-locale)
 - [Contributions](#contributions)
+- [Publier une nouvelle version](#publier-une-nouvelle-version)
 - [TODO](#todo)
 - [Archives](#archives)
 
@@ -61,7 +62,7 @@ $ echo '--force-exclusion' > .rubocop
 ### 3. Config Overcommit
 
 ```bash
-$ curl https://raw.githubusercontent.com/honestica/lifen-ruby-style/master/default_overcommit.yml > .overcommit_test.yml
+$ curl https://raw.githubusercontent.com/honestica/lifen-ruby-style/master/default_overcommit.yml > .overcommit.yml
 ```
 
 ### 4. Installer les dépendances
@@ -123,17 +124,20 @@ $ bundle exec overcommit --install
 - Paramétrage recommandé pour l'auto-formatting :
 
   ```json
+    "editor.formatOnPaste": true,  // global auto-format code when pasting
+    "editor.formatOnSave": true,   // global auto-format code when saving
     "[ruby]": {
-      "editor.formatOnPaste": true,       // auto-format ruby code when pasting
-      "editor.formatOnSave": true         // auto-format ruby code when saving
+      "editor.formatOnSave": false // disable auto-format on save only for ruby files
     },
   ```
 
   ou
 
   ```json
-    "editor.formatOnPaste": true,  // auto-format code when pasting
-    "editor.formatOnSave": true,   // auto-format code when saving
+    "[ruby]": {
+      "editor.formatOnPaste": true,       // auto-format ruby code when pasting
+      "editor.formatOnSave": false         // auto-format ruby code when saving
+    },
   ```
 
 ### 3. Bonus - Autres extensions utiles :
@@ -267,6 +271,11 @@ https://github.com/dense-analysis/ale
   - Si je veux bypasser les hooks (déconseillé) : `OVERCOMMIT_DISABLE=1 git commit ...`
 - Je push (pas de hooks de pre-push paramétrés pour l'instant)
 
+Attention :
+
+- si vous faites "Ctrl+C" pendant qu'un hook overcommit tourne, overcommit fait un `git stash save` et vous pouvez retrouver ce qui a disparu avec `git stast list / show / pop`
+- dans le même registre, ne faites pas de `watch .git/hooks/pre-commit`, si vous éditez vos fichiers dans une autre fenêtre, overcommit va régulièrement stasher vos modifs, et vous allez tourner en rond à éditer des fichiers qui changent sous vos pieds.
+
 ### Commandes Rubocop
 
 Pour de la vérification :
@@ -360,14 +369,19 @@ Vous allez vouloir tester localement les modifications de règle de style que vo
 
 **ATTENTION : La configuration d'overcommit n'est pas liée automatiquement au projet, il faut la copier/coller dans le projet cible après avoir effectué des changements.**
 
+## Publier une nouvelle version
+
+Suivre les instructions du [guide de release](RELEASING.md).
+
 ## TODO
 
-[ ] Ajouter le hook de pre-commit au CI
-[ ] Ajouter un hook de pre-push ?
-[ ] Ajouter `fasterer` si pertinent
-[ ] Ajouter `reek` si pertinent
-[ ] Ajouter `slim_lint` si pertinent
-[ ] Ajouter `scss` si pertinent
+- [ ] Ajouter rubocop-daemon (arriver à faire une bonne intégration aux IDE)
+- [ ] Ajouter le hook de pre-commit au CI
+- [ ] Ajouter un hook de pre-push ?
+- [ ] Ajouter `fasterer` si pertinent
+- [ ] Ajouter `reek` si pertinent
+- [ ] Ajouter `slim_lint` si pertinent
+- [ ] Ajouter `scss` si pertinent
 
 ## Archives
 

data/default_rubocop.yml

@@ -5,6 +5,7 @@ require:
 
 AllCops:
   DisplayStyleGuide: true
+  NewCops: enable
   TargetRubyVersion: 2.5
   Exclude:
     - "test/**/*.rb"
@@ -16,14 +17,6 @@ AllCops:
     - "spec/rails_helper.rb"
     - "node_modules/**/*"
 
-# Waiting for default config
-Style/HashEachMethods:
-  Enabled: true
-Style/HashTransformKeys:
-  Enabled: true
-Style/HashTransformValues:
-  Enabled: true
-
 Layout/ArgumentAlignment:
   EnforcedStyle: with_fixed_indentation
 
@@ -87,6 +80,10 @@ Metrics/PerceivedComplexity:
 Naming/MemoizedInstanceVariableName:
   EnforcedStyleForLeadingUnderscores: required
 
+RSpec/DescribeClass:
+  Exclude:
+    - spec/lib/tasks/deployment/**/*
+
 Style/BlockDelimiters:
   EnforcedStyle: braces_for_chaining
 
@@ -96,6 +93,18 @@ Style/ClassAndModuleChildren:
 Style/Documentation:
   Enabled: false
 
+Style/TrailingCommaInArguments:
+  Enabled: true
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInArrayLiteral:
+  Enabled: true
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInHashLiteral:
+  Enabled: true
+  EnforcedStyleForMultiline: comma
+
 Style/IfUnlessModifier:
   Enabled: true
   AutoCorrect: false
@@ -132,15 +141,24 @@ RSpec/AnyInstance:
 RSpec/DescribedClass:
   EnforcedStyle: explicit
 
+RSpec/EmptyExampleGroup:
+  Enabled: false
+
 RSpec/ExampleLength:
   Enabled: false
 
 RSpec/HookArgument:
   EnforcedStyle: each
 
+RSpec/LetSetup:
+  Enabled: false
+
 RSpec/MultipleExpectations:
   Enabled: false
 
+RSpec/MultipleMemoizedHelpers:
+  Enabled: false
+
 RSpec/NestedGroups:
   Max: 5
 

data/lib/lifen/ruby_style/version.rb

@@ -3,7 +3,7 @@
 module Lifen
   module RubyStyle
 
-    VERSION = '1.0.3'
+    VERSION = '1.0.8'
 
   end
 end

data/lifen-ruby-style.gemspec

@@ -30,7 +30,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'overcommit'
-  spec.add_dependency 'rubocop', '~> 0.8'
+  spec.add_dependency 'rubocop', '~> 0.88'
   spec.add_dependency 'rubocop-performance', '~> 1.5'
   spec.add_dependency 'rubocop-rails', '~> 2.4'
   spec.add_dependency 'rubocop-rspec', '~> 1.38'

data/style_guide.md

@@ -619,6 +619,12 @@ before(:each) do
 end
 ```
 
+#### LetSetup
+
+```yaml
+Enabled: false //  does not check unreferenced let! calls being used for test setup
+```
+
 #### MultipleExpectations
 
 ```yaml

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lifen-ruby-style
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.8
 platform: ruby
 authors:
 - Team Lifen
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-31 00:00:00.000000000 Z
+date: 2020-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: overcommit
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '0.88'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '0.88'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement