puppet-lint 2.0.1 → 2.0.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: e6c8e66a832c7a2cb2f29ed9bcba97b1f7807de3
|
4
|
+
data.tar.gz: 480e6c20a669db2668c3366f97c6b1133acf8217
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: ba2e39b0d3cea77c78d46fd33dd7d4bacb17cfcc9780e941bd9f623b981acbcc228e751ee660a8875769e1120696955c232687083054d8cb7bc61438c2901d59
|
7
|
+
data.tar.gz: 5b50d449286742482856f009ee9022c3296a7dd317cb1a2403c7631c9a4c9d084212ebb58442e3d4e92d431f9101205711a59f9624c2dd94daf854885386380f
|
data/CHANGELOG.md
CHANGED
@@ -1,5 +1,19 @@
|
|
1
1
|
# Change Log
|
2
2
|
|
3
|
+
## [2.0.2](https://github.com/rodjek/puppet-lint/tree/2.0.2) (2016-08-19)
|
4
|
+
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.0.1...2.0.2)
|
5
|
+
|
6
|
+
**Closed issues:**
|
7
|
+
|
8
|
+
- Start a CHANGELOG, make updates part of the build workflow [\#479](https://github.com/rodjek/puppet-lint/issues/479)
|
9
|
+
- Nested future scope blocks lose local variables from parent scopes. [\#456](https://github.com/rodjek/puppet-lint/issues/456)
|
10
|
+
- block-local variables \(|$x|\) don't properly get recognized when used as arrays or hashes [\#450](https://github.com/rodjek/puppet-lint/issues/450)
|
11
|
+
- Option to choose version of the puppet style guide [\#190](https://github.com/rodjek/puppet-lint/issues/190)
|
12
|
+
|
13
|
+
**Merged pull requests:**
|
14
|
+
|
15
|
+
- Revert "Bugfix: properly handling $gronk-$grouik with --fix \(fix \#442\)" [\#535](https://github.com/rodjek/puppet-lint/pull/535) ([rnelson0](https://github.com/rnelson0))
|
16
|
+
|
3
17
|
## [2.0.1](https://github.com/rodjek/puppet-lint/tree/2.0.1) (2016-08-18)
|
4
18
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.0.0...2.0.1)
|
5
19
|
|
@@ -53,10 +67,13 @@
|
|
53
67
|
- Catch code outside of class or define block. [\#220](https://github.com/rodjek/puppet-lint/issues/220)
|
54
68
|
- Catch global code outside of node blocks [\#160](https://github.com/rodjek/puppet-lint/issues/160)
|
55
69
|
- Add a warning for resources outside of a class [\#69](https://github.com/rodjek/puppet-lint/issues/69)
|
70
|
+
- Release a new build, 2.0.1 [\#519](https://github.com/rodjek/puppet-lint/issues/519)
|
71
|
+
- Puppet-Lint 1.1.0 unhandled issue causing exit during -f [\#366](https://github.com/rodjek/puppet-lint/issues/366)
|
56
72
|
|
57
73
|
**Merged pull requests:**
|
58
74
|
|
59
75
|
- Continue supporting Ruby \< 2 [\#529](https://github.com/rodjek/puppet-lint/pull/529) ([ghoneycutt](https://github.com/ghoneycutt))
|
76
|
+
- Release 2.0.1 [\#528](https://github.com/rodjek/puppet-lint/pull/528) ([rnelson0](https://github.com/rnelson0))
|
60
77
|
- Use underscores instead of CamelCase in example [\#527](https://github.com/rodjek/puppet-lint/pull/527) ([ghoneycutt](https://github.com/ghoneycutt))
|
61
78
|
- new linter for top\_scope\_facts [\#526](https://github.com/rodjek/puppet-lint/pull/526) ([mmckinst](https://github.com/mmckinst))
|
62
79
|
- \(GH462\) Multi-line comments can now be fixed. [\#525](https://github.com/rodjek/puppet-lint/pull/525) ([rnelson0](https://github.com/rnelson0))
|
@@ -654,4 +671,4 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
654
671
|
## [0.0.1](https://github.com/rodjek/puppet-lint/tree/0.0.1) (2011-08-15)
|
655
672
|
|
656
673
|
|
657
|
-
\* *This Change Log was automatically generated by [github_changelog_generator](https://github.com/skywinder/Github-Changelog-Generator)*
|
674
|
+
\* *This Change Log was automatically generated by [github_changelog_generator](https://github.com/skywinder/Github-Changelog-Generator)*
|
data/README.md
CHANGED
@@ -174,11 +174,6 @@ If you find a bug in puppet-lint or its results, please create an issue in the
|
|
174
174
|
[repo issues tracker](https://github.com/rodjek/puppet-lint/issues/). Bonus
|
175
175
|
points will be awarded if you also include a patch that fixes the issue.
|
176
176
|
|
177
|
-
## Executing puppet-lint tests suite
|
178
|
-
|
179
|
-
bundle exec rspec [spec/puppet-lint/a_single_test.rb]
|
180
|
-
|
181
|
-
|
182
177
|
## Thank You
|
183
178
|
|
184
179
|
Many thanks to the following people for contributing to puppet-lint
|
data/lib/puppet-lint/lexer.rb
CHANGED
@@ -170,7 +170,7 @@ class PuppetLint
|
|
170
170
|
end
|
171
171
|
|
172
172
|
unless found
|
173
|
-
if var_name = chunk[/\A\$((::)?([\w]+::)*[\w]+(\[.+?\])*)/, 1]
|
173
|
+
if var_name = chunk[/\A\$((::)?([\w-]+::)*[\w-]+(\[.+?\])*)/, 1]
|
174
174
|
length = var_name.size + 1
|
175
175
|
tokens << new_token(:VARIABLE, var_name, length)
|
176
176
|
|
@@ -346,7 +346,7 @@ class PuppetLint
|
|
346
346
|
tokens << new_token(:DQMID, value, value.size, :line => line, :column => token_column)
|
347
347
|
end
|
348
348
|
if ss.scan(/\{/).nil?
|
349
|
-
var_name = ss.scan(/(::)?([\w]+::)*[\w]+/)
|
349
|
+
var_name = ss.scan(/(::)?([\w-]+::)*[\w-]+/)
|
350
350
|
if var_name.nil?
|
351
351
|
token_column = column + ss.pos - 1
|
352
352
|
tokens << new_token(:DQMID, "$", 1, :line => line, :column => token_column)
|
@@ -7,7 +7,7 @@ PuppetLint.new_check(:variable_contains_dash) do
|
|
7
7
|
tokens.select { |r|
|
8
8
|
VARIABLE_DASH_TYPES.include? r.type
|
9
9
|
}.each do |token|
|
10
|
-
if
|
10
|
+
if token.value.gsub(/\[.+?\]/, '').match(/-/)
|
11
11
|
notify :warning, {
|
12
12
|
:message => 'variable contains a dash',
|
13
13
|
:line => token.line,
|
data/lib/puppet-lint/version.rb
CHANGED
@@ -39,10 +39,10 @@ describe 'variables_not_enclosed' do
|
|
39
39
|
end
|
40
40
|
|
41
41
|
context 'variable not enclosed in {}' do
|
42
|
-
let(:code) { '" $gronk
|
42
|
+
let(:code) { '" $gronk"' }
|
43
43
|
|
44
44
|
it 'should only detect a single problem' do
|
45
|
-
expect(problems).to have(
|
45
|
+
expect(problems).to have(1).problem
|
46
46
|
end
|
47
47
|
|
48
48
|
it 'should fix the manifest' do
|
@@ -50,7 +50,7 @@ describe 'variables_not_enclosed' do
|
|
50
50
|
end
|
51
51
|
|
52
52
|
it 'should enclose the variable in braces' do
|
53
|
-
expect(manifest).to eq('" ${gronk}
|
53
|
+
expect(manifest).to eq('" ${gronk}"')
|
54
54
|
end
|
55
55
|
end
|
56
56
|
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: puppet-lint
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 2.0.
|
4
|
+
version: 2.0.2
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Tim Sharpe
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2016-08-
|
11
|
+
date: 2016-08-19 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: github_changelog_generator
|