datarockets-style 0.2.0 → 0.3.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.github/ISSUE_TEMPLATE/new-issue.md +17 -0
- data/.github/ISSUE_TEMPLATE/update-dependencies.md +16 -0
- data/.github/PULL_REQUEST_TEMPLATE.md +10 -0
- data/CHANGELOG.md +31 -15
- data/Gemfile.lock +5 -5
- data/README.md +1 -1
- data/RELEASING.md +7 -7
- data/STYLE_GUIDE.md +42 -10
- data/{default.yml → config/default.yml} +11 -0
- data/datarockets-style.gemspec +2 -2
- data/lib/datarockets/style/version.rb +1 -1
- metadata +10 -7
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 0dbec2a9995d6607b63dafed5a123bfdb8704e3a
|
4
|
+
data.tar.gz: 50bee07693998a77cf012e2cff94905095e111fc
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: b6e5844c8bee41e0c621cf732808284d0f7596f1949c65396ef4dcbbccc830141209f3b1e8865d9e9b59b38ff5e2c36dd8dfe35f38cddfaad74a9f87101d9b33
|
7
|
+
data.tar.gz: '096a00c0585458c1fd3ca87f0b116db3b90588c368e3a4c86dc76f253932516c3b80dd8099f8e09dc810ee4dbc8b14c5b9a7c5e66f0b378245f1eec9174faef9'
|
@@ -0,0 +1,17 @@
|
|
1
|
+
---
|
2
|
+
name: New issue or feature request
|
3
|
+
about: Suggest a new feature or enhancement, ask a question, report bug, etc.
|
4
|
+
title: "Title goes here"
|
5
|
+
labels: ''
|
6
|
+
assignees: ''
|
7
|
+
|
8
|
+
---
|
9
|
+
|
10
|
+
**Is your feature request related to a problem? Please describe.**
|
11
|
+
A clear and concise description of what the problem is.
|
12
|
+
|
13
|
+
**Additional context.**
|
14
|
+
Add any other context or screenshots about the feature request here.
|
15
|
+
|
16
|
+
**Add label.**
|
17
|
+
Add label for easier navigation by issues and understanding your expectations.
|
@@ -0,0 +1,16 @@
|
|
1
|
+
---
|
2
|
+
name: Update Dependencies
|
3
|
+
about: Updating dependencies issue
|
4
|
+
title: "Update *dependency* to *version*"
|
5
|
+
labels: dependencies
|
6
|
+
assignees: ''
|
7
|
+
|
8
|
+
---
|
9
|
+
|
10
|
+
Link to the new release: *link*
|
11
|
+
|
12
|
+
To-Do list:
|
13
|
+
|
14
|
+
- [ ] update dependencies to the latest version
|
15
|
+
- [ ] investigate how it changes and affect our current styles and cops
|
16
|
+
- [ ] check the required ruby version
|
@@ -0,0 +1,10 @@
|
|
1
|
+
Before you submit a pull request, please make sure you have to follow:
|
2
|
+
|
3
|
+
- [ ] read and know items from the [Contributing Guide](CONTRIBUTING.md#pull-requests)
|
4
|
+
- [ ] add a description of the problem you're trying to solve (short summary from related issue)
|
5
|
+
- [ ] verified that cops are ordered by alphabet
|
6
|
+
- [ ] add a note to the style guide docs (if it needs)
|
7
|
+
- [ ] add a note to the changelog file
|
8
|
+
- [ ] the commit message contains the number of the related issue (if it presents)
|
9
|
+
and word `Fix` if this PR closes related issue
|
10
|
+
- [ ] squash all commits before submitting to review
|
data/CHANGELOG.md
CHANGED
@@ -1,15 +1,30 @@
|
|
1
1
|
# Change Log
|
2
2
|
|
3
|
-
The format is described in [Contributing notes](CONTRIBUTING.md#changelog-entry-format)
|
3
|
+
The format is described in [Contributing notes](CONTRIBUTING.md#changelog-entry-format).
|
4
4
|
|
5
5
|
## master (unreleased)
|
6
6
|
|
7
|
+
## 0.3.0 (2019-08-02)
|
8
|
+
|
9
|
+
### Changed
|
10
|
+
|
11
|
+
* Update rubocop to `0.74.0`. ([@r.dubrovsky][])
|
12
|
+
* Update rubocop-rspec to `1.34.1`. ([@r.dubrovsky][])
|
13
|
+
* Move shared rubocop config into `config` directory. ([@r.dubrovsky][])
|
14
|
+
* Allow writing empty methods in two lines. ([@v.kuzmik][])
|
15
|
+
* Disable `Style/FrozenStringLiteralComment` cop by default. ([@r.dubrovsky][])
|
16
|
+
|
17
|
+
### Fixed
|
18
|
+
|
19
|
+
* [#4222](https://github.com/rubocop-hq/rubocop/issues/4222): Disable `Lint/AmbiguousBlockAssociation` for Rspec directory. ([@r.dubrovsky][])
|
20
|
+
* [#65](https://github.com/datarockets/datarockets-style/issues/65): Exclude `node_modules` from rubocop scope. ([@r.dubrovsky][])
|
21
|
+
|
7
22
|
## 0.2.0 (2019-07-17)
|
8
23
|
|
9
24
|
### Changed
|
10
25
|
|
11
26
|
* Update rubocop to `0.73.0`. ([@r.dubrovsky][])
|
12
|
-
* Use prefered variable name in rescued exceptions (cop: `Naming/RescuedExceptionsVariableName`) ([@ula][])
|
27
|
+
* Use prefered variable name in rescued exceptions (cop: `Naming/RescuedExceptionsVariableName`). ([@ula][])
|
13
28
|
* Disable `RSpec/ImplicitSubject` cop for rspec files. ([@r.dubrovsky][])
|
14
29
|
|
15
30
|
## 0.1.0 (2019-06-27)
|
@@ -17,24 +32,25 @@ The format is described in [Contributing notes](CONTRIBUTING.md#changelog-entry-
|
|
17
32
|
### Added
|
18
33
|
|
19
34
|
* Base config with community rules and some basic override rules. ([@r.dubrovsky][], [@aleks][])
|
20
|
-
* Config is based on rubocop version 0.72.0 ([@r.dubrovsky][])
|
21
|
-
* Enable `rubocop-spec` cops by default ([@r.dubrovsky][])
|
22
|
-
* Enable `Bundler/DuplicatedGem` cop by default ([@r.dubrovsky][])
|
23
|
-
* Enable `Bundler/OrderedGems` cop with allowing ordering by groups ([@r.dubrovsky][])
|
35
|
+
* Config is based on rubocop version 0.72.0. ([@r.dubrovsky][])
|
36
|
+
* Enable `rubocop-spec` cops by default. ([@r.dubrovsky][])
|
37
|
+
* Enable `Bundler/DuplicatedGem` cop by default. ([@r.dubrovsky][])
|
38
|
+
* Enable `Bundler/OrderedGems` cop with allowing ordering by groups. ([@r.dubrovsky][])
|
24
39
|
|
25
40
|
|
26
41
|
### Changed
|
27
42
|
|
28
|
-
* Change the limit for size of line to 120 symbols ([@r.dubrovsky][])
|
29
|
-
* Disable `Metrics/BlockLength` cop for rspec files ([@r.dubrovsky][])
|
30
|
-
* Exclude rubocop checking for some config directories ([@r.dubrovsky][])
|
31
|
-
* Enable prafering double quotes for string literals ([@r.dubrovsky][])
|
32
|
-
* Do not add spaces beetwen hash literal braces (cop `Layout/SpaceInsideHashLiteralBraces`) ([@r.dubrovsky][])
|
33
|
-
* Prefer normal style for `Layout/IndentationConsistency` cop for Rails apps too ([@r.dubrovsky][])
|
34
|
-
* Change style to `variable` for `Layout/EndAlignment` cop ([@r.dubrovsky][])
|
35
|
-
* Change style to `with_fixed_indentation` with indentation width 2 for `Layout/AlignParameter` cop ([@r.dubrovsky][])
|
36
|
-
* Always ingone hash aligning for key word arguments (cop: `Layout/AlignHash`) ([@r.dubrovsky][])
|
43
|
+
* Change the limit for size of line to 120 symbols. ([@r.dubrovsky][])
|
44
|
+
* Disable `Metrics/BlockLength` cop for rspec files. ([@r.dubrovsky][])
|
45
|
+
* Exclude rubocop checking for some config directories. ([@r.dubrovsky][])
|
46
|
+
* Enable prafering double quotes for string literals. ([@r.dubrovsky][])
|
47
|
+
* Do not add spaces beetwen hash literal braces (cop `Layout/SpaceInsideHashLiteralBraces`). ([@r.dubrovsky][])
|
48
|
+
* Prefer normal style for `Layout/IndentationConsistency` cop for Rails apps too. ([@r.dubrovsky][])
|
49
|
+
* Change style to `variable` for `Layout/EndAlignment` cop. ([@r.dubrovsky][])
|
50
|
+
* Change style to `with_fixed_indentation` with indentation width 2 for `Layout/AlignParameter` cop. ([@r.dubrovsky][])
|
51
|
+
* Always ingone hash aligning for key word arguments. (cop: `Layout/AlignHash`) ([@r.dubrovsky][])
|
37
52
|
|
38
53
|
[@r.dubrovsky]: https://github.com/roman-dubrovsky
|
39
54
|
[@aleks]: https://github.com/AleksSenkou
|
40
55
|
[@ula]: https://github.com/lazycoder9
|
56
|
+
[@v.kuzmik]: https://github.com/TheBlackArroVV/
|
data/Gemfile.lock
CHANGED
@@ -1,9 +1,9 @@
|
|
1
1
|
PATH
|
2
2
|
remote: .
|
3
3
|
specs:
|
4
|
-
datarockets-style (0.
|
5
|
-
rubocop (~> 0.
|
6
|
-
rubocop-rspec (~> 1.
|
4
|
+
datarockets-style (0.3.0)
|
5
|
+
rubocop (~> 0.74.0)
|
6
|
+
rubocop-rspec (~> 1.34)
|
7
7
|
|
8
8
|
GEM
|
9
9
|
remote: https://rubygems.org/
|
@@ -15,14 +15,14 @@ GEM
|
|
15
15
|
ast (~> 2.4.0)
|
16
16
|
rainbow (3.0.0)
|
17
17
|
rake (10.5.0)
|
18
|
-
rubocop (0.
|
18
|
+
rubocop (0.74.0)
|
19
19
|
jaro_winkler (~> 1.5.1)
|
20
20
|
parallel (~> 1.10)
|
21
21
|
parser (>= 2.6)
|
22
22
|
rainbow (>= 2.2.2, < 4.0)
|
23
23
|
ruby-progressbar (~> 1.7)
|
24
24
|
unicode-display_width (>= 1.4.0, < 1.7)
|
25
|
-
rubocop-rspec (1.
|
25
|
+
rubocop-rspec (1.34.1)
|
26
26
|
rubocop (>= 0.60.0)
|
27
27
|
ruby-progressbar (1.10.1)
|
28
28
|
unicode-display_width (1.6.0)
|
data/README.md
CHANGED
data/RELEASING.md
CHANGED
@@ -1,17 +1,17 @@
|
|
1
1
|
# Releasing
|
2
2
|
|
3
3
|
* Update `version.rb` file accordingly.
|
4
|
-
* Update `CHANGELOG.md` file
|
5
|
-
*
|
6
|
-
*
|
7
|
-
* Update the release notes on GitHub.com
|
8
|
-
* Build and publish:
|
4
|
+
* Update `CHANGELOG.md` file.
|
5
|
+
* Update `Gemfile.lock` file via running `bundle install`.
|
6
|
+
* Build and publish (and verify that everything ok):
|
9
7
|
|
10
8
|
```bash
|
11
9
|
bundle exec rake build
|
12
|
-
gem push *
|
10
|
+
gem push *built_file*
|
13
11
|
```
|
14
|
-
|
12
|
+
* Tag the release: `git tag vVERSION`.
|
13
|
+
* Push changes: `git push --tags`.
|
14
|
+
* Update the release notes on GitHub.com.
|
15
15
|
* Announce the new release,
|
16
16
|
making sure to say "thank you" to the contributors
|
17
17
|
who helped shape this version!
|
data/STYLE_GUIDE.md
CHANGED
@@ -72,33 +72,39 @@ do_something(foo: 1,
|
|
72
72
|
The parameters on a multi-line method call or definition are aligning by an indentation level.
|
73
73
|
<sup>[[link](#style-parameters-aligning)]</sup>
|
74
74
|
|
75
|
+
**Recommended:** write each parameter on the separate line.
|
76
|
+
|
75
77
|
```ruby
|
76
|
-
#
|
78
|
+
# bad
|
77
79
|
|
78
|
-
def foo(bar,
|
79
|
-
|
80
|
+
def foo(bar, baz,
|
81
|
+
kek, lol)
|
80
82
|
123
|
81
83
|
end
|
82
84
|
|
85
|
+
# bad
|
86
|
+
|
83
87
|
def foo(
|
84
88
|
bar,
|
85
|
-
|
86
|
-
)
|
89
|
+
baz)
|
87
90
|
123
|
88
91
|
end
|
89
92
|
|
90
|
-
#
|
93
|
+
# good
|
91
94
|
|
92
|
-
def foo(bar,
|
93
|
-
|
95
|
+
def foo(bar, baz,
|
96
|
+
kek, lol)
|
94
97
|
123
|
95
98
|
end
|
96
99
|
|
97
|
-
#
|
100
|
+
# better
|
98
101
|
|
99
102
|
def foo(
|
100
103
|
bar,
|
101
|
-
|
104
|
+
baz,
|
105
|
+
kek,
|
106
|
+
lol
|
107
|
+
)
|
102
108
|
123
|
103
109
|
end
|
104
110
|
```
|
@@ -240,6 +246,32 @@ rescue MyException => error
|
|
240
246
|
# do something
|
241
247
|
end
|
242
248
|
```
|
249
|
+
|
250
|
+
|
251
|
+
* <a name="style-empty-method"></a>
|
252
|
+
Write empty methods in an expanded way.
|
253
|
+
<sup>[[link](#style-empty-method)]</sup>
|
254
|
+
|
255
|
+
```ruby
|
256
|
+
# bad
|
257
|
+
def foo(bar); end
|
258
|
+
|
259
|
+
def self.foo(bar); end
|
260
|
+
|
261
|
+
# good
|
262
|
+
def foo(bar)
|
263
|
+
end
|
264
|
+
|
265
|
+
def self.foo(bar)
|
266
|
+
end
|
267
|
+
```
|
268
|
+
|
269
|
+
* <a name="style-magic-link"></a>
|
270
|
+
There are not any requried rules for `frozen_string_literal` magic url.
|
271
|
+
Set up [this cop](https://rubocop.readthedocs.io/en/latest/cops_style/#stylefrozenstringliteralcomment) depends on the project.
|
272
|
+
So set up it on the local rubocop config manually.
|
273
|
+
<sup>[[link](#style-magic-link)]</sup>
|
274
|
+
|
243
275
|
## Rspec
|
244
276
|
|
245
277
|
* <a name="rspec-betterrspec"></a>
|
@@ -4,6 +4,7 @@ AllCops:
|
|
4
4
|
- "bin/*"
|
5
5
|
- "lib/tasks/**/*"
|
6
6
|
- "config/**/*"
|
7
|
+
- "node_modules/**/*"
|
7
8
|
- "vendor/**/*"
|
8
9
|
- "app/views/**/*"
|
9
10
|
- "config.ru"
|
@@ -34,6 +35,10 @@ Layout/SpaceInsideHashLiteralBraces:
|
|
34
35
|
EnforcedStyle: no_space
|
35
36
|
EnforcedStyleForEmptyBraces: no_space
|
36
37
|
|
38
|
+
Lint/AmbiguousBlockAssociation:
|
39
|
+
Exclude:
|
40
|
+
- "spec/**/*"
|
41
|
+
|
37
42
|
Metrics/BlockLength:
|
38
43
|
Exclude:
|
39
44
|
- "spec/**/*"
|
@@ -47,5 +52,11 @@ Naming/RescuedExceptionsVariableName:
|
|
47
52
|
RSpec/ImplicitSubject:
|
48
53
|
Enabled: false
|
49
54
|
|
55
|
+
Style/EmptyMethod:
|
56
|
+
EnforcedStyle: expanded
|
57
|
+
|
58
|
+
Style/FrozenStringLiteralComment:
|
59
|
+
Enabled: false
|
60
|
+
|
50
61
|
Style/StringLiterals:
|
51
62
|
EnforcedStyle: double_quotes
|
data/datarockets-style.gemspec
CHANGED
@@ -30,8 +30,8 @@ Gem::Specification.new do |spec|
|
|
30
30
|
spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
|
31
31
|
spec.require_paths = ["lib"]
|
32
32
|
|
33
|
-
spec.add_dependency "rubocop", "~> 0.
|
34
|
-
spec.add_dependency "rubocop-rspec", "~> 1.
|
33
|
+
spec.add_dependency "rubocop", "~> 0.74.0"
|
34
|
+
spec.add_dependency "rubocop-rspec", "~> 1.34"
|
35
35
|
|
36
36
|
spec.add_development_dependency "bundler", "~> 1.16"
|
37
37
|
spec.add_development_dependency "rake", "~> 10.0"
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: datarockets-style
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.
|
4
|
+
version: 0.3.0
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Roman Dubrovsky
|
8
8
|
autorequire:
|
9
9
|
bindir: exe
|
10
10
|
cert_chain: []
|
11
|
-
date: 2019-
|
11
|
+
date: 2019-08-02 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: rubocop
|
@@ -16,28 +16,28 @@ dependencies:
|
|
16
16
|
requirements:
|
17
17
|
- - "~>"
|
18
18
|
- !ruby/object:Gem::Version
|
19
|
-
version: 0.
|
19
|
+
version: 0.74.0
|
20
20
|
type: :runtime
|
21
21
|
prerelease: false
|
22
22
|
version_requirements: !ruby/object:Gem::Requirement
|
23
23
|
requirements:
|
24
24
|
- - "~>"
|
25
25
|
- !ruby/object:Gem::Version
|
26
|
-
version: 0.
|
26
|
+
version: 0.74.0
|
27
27
|
- !ruby/object:Gem::Dependency
|
28
28
|
name: rubocop-rspec
|
29
29
|
requirement: !ruby/object:Gem::Requirement
|
30
30
|
requirements:
|
31
31
|
- - "~>"
|
32
32
|
- !ruby/object:Gem::Version
|
33
|
-
version: '1.
|
33
|
+
version: '1.34'
|
34
34
|
type: :runtime
|
35
35
|
prerelease: false
|
36
36
|
version_requirements: !ruby/object:Gem::Requirement
|
37
37
|
requirements:
|
38
38
|
- - "~>"
|
39
39
|
- !ruby/object:Gem::Version
|
40
|
-
version: '1.
|
40
|
+
version: '1.34'
|
41
41
|
- !ruby/object:Gem::Dependency
|
42
42
|
name: bundler
|
43
43
|
requirement: !ruby/object:Gem::Requirement
|
@@ -73,6 +73,9 @@ executables: []
|
|
73
73
|
extensions: []
|
74
74
|
extra_rdoc_files: []
|
75
75
|
files:
|
76
|
+
- ".github/ISSUE_TEMPLATE/new-issue.md"
|
77
|
+
- ".github/ISSUE_TEMPLATE/update-dependencies.md"
|
78
|
+
- ".github/PULL_REQUEST_TEMPLATE.md"
|
76
79
|
- ".gitignore"
|
77
80
|
- CHANGELOG.md
|
78
81
|
- CODE_OF_CONDUCT.md
|
@@ -86,8 +89,8 @@ files:
|
|
86
89
|
- STYLE_GUIDE.md
|
87
90
|
- bin/console
|
88
91
|
- bin/setup
|
92
|
+
- config/default.yml
|
89
93
|
- datarockets-style.gemspec
|
90
|
-
- default.yml
|
91
94
|
- lib/datarockets/style.rb
|
92
95
|
- lib/datarockets/style/version.rb
|
93
96
|
homepage: https://github.com/datarockets/datarockets-style
|