puppet-lint 2.4.2 → 2.5.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/CHANGELOG.md +112 -96
- data/lib/puppet-lint.rb +1 -1
- data/lib/puppet-lint/lexer.rb +7 -5
- data/lib/puppet-lint/lexer/string_slurper.rb +14 -3
- data/lib/puppet-lint/lexer/token.rb +6 -0
- data/lib/puppet-lint/plugins/check_classes/parameter_order.rb +12 -1
- data/lib/puppet-lint/plugins/check_documentation/documentation.rb +4 -0
- data/lib/puppet-lint/plugins/check_resources/ensure_first_param.rb +1 -1
- data/lib/puppet-lint/tasks/puppet-lint.rb +14 -0
- data/lib/puppet-lint/tasks/release_test.rb +3 -1
- data/lib/puppet-lint/version.rb +1 -1
- data/spec/puppet-lint/bin_spec.rb +8 -8
- data/spec/puppet-lint/lexer/string_slurper_spec.rb +34 -0
- data/spec/puppet-lint/plugins/check_classes/parameter_order_spec.rb +18 -0
- data/spec/puppet-lint/plugins/check_documentation/documentation_spec.rb +18 -0
- data/spec/puppet-lint/plugins/check_strings/double_quoted_strings_spec.rb +5 -5
- data/spec/puppet-lint/plugins/check_variables/variable_is_lowercase_spec.rb +28 -0
- data/spec/spec_helper.rb +7 -5
- metadata +9 -16
- data/.gitignore +0 -12
- data/.rspec +0 -2
- data/.rubocop.yml +0 -74
- data/.rubocop_todo.yml +0 -89
- data/.travis.yml +0 -26
- data/Gemfile +0 -40
- data/Rakefile +0 -44
- data/appveyor.yml +0 -35
- data/puppet-lint.gemspec +0 -19
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 1009204f664cf2b2b110034e180ebebf26fb25731449a72e24a2cc5e890de99b
|
4
|
+
data.tar.gz: '09eebeffc2425b6a3984db1653527a16b9af06b71284b35612ed8d58af94a4ff'
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 129ac3f77fd825c75b0fd081edf68b54a7409b642a527e097aba0f44695858b0872ff0839675005b11b3da39e8faa1b2df7ac35a82923a6cd2b645ac3c7aac96
|
7
|
+
data.tar.gz: c86f851bcd4089d805aca6e91785f6d7f8e49dc55080e8a35a98f070b1d42039b5cc69909fe2002446029fcb0d38a785da77b4872378e8850ad28d82c68f6e01
|
data/CHANGELOG.md
CHANGED
@@ -1,12 +1,58 @@
|
|
1
1
|
# Changelog
|
2
2
|
|
3
|
+
## [2.5.0](https://github.com/rodjek/puppet-lint/tree/2.5.0) (2021-07-23)
|
4
|
+
|
5
|
+
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.4.2...2.5.0)
|
6
|
+
|
7
|
+
**Implemented enhancements:**
|
8
|
+
|
9
|
+
- Include check name in log output by default [\#883](https://github.com/rodjek/puppet-lint/pull/883) ([usev6](https://github.com/usev6))
|
10
|
+
|
11
|
+
**Fixed bugs:**
|
12
|
+
|
13
|
+
- Array operations on parameters confuse puppet-lint [\#930](https://github.com/rodjek/puppet-lint/issues/930)
|
14
|
+
- Wrong warrning: "variable contains an uppercase" [\#929](https://github.com/rodjek/puppet-lint/issues/929)
|
15
|
+
- Erroneous "WARNING: indentation of =\> is not properly aligned" since 2.4.0 [\#912](https://github.com/rodjek/puppet-lint/issues/912)
|
16
|
+
- NoMethodError: undefined method `next\_token\_of' for nil:NilClass [\#906](https://github.com/rodjek/puppet-lint/issues/906)
|
17
|
+
- Lint failure on reduce function [\#869](https://github.com/rodjek/puppet-lint/issues/869)
|
18
|
+
- lint failure on 'ensure' word in dsc\_lite module [\#866](https://github.com/rodjek/puppet-lint/issues/866)
|
19
|
+
- Content of heredocs should not be checked [\#851](https://github.com/rodjek/puppet-lint/issues/851)
|
20
|
+
- Whoops! It looks like puppet-lint has encountered an error that it doesn't [\#839](https://github.com/rodjek/puppet-lint/issues/839)
|
21
|
+
|
22
|
+
**Closed issues:**
|
23
|
+
|
24
|
+
- heredoc prematurely terminated [\#940](https://github.com/rodjek/puppet-lint/issues/940)
|
25
|
+
- Heredoc syntax error [\#935](https://github.com/rodjek/puppet-lint/issues/935)
|
26
|
+
- Lint for ' [\#927](https://github.com/rodjek/puppet-lint/issues/927)
|
27
|
+
- puppet-lint-class\_parameter-check fails on voxpupuli/collectd [\#926](https://github.com/rodjek/puppet-lint/issues/926)
|
28
|
+
- puppet-lint error [\#917](https://github.com/rodjek/puppet-lint/issues/917)
|
29
|
+
- Improper "unquoted node name found" [\#904](https://github.com/rodjek/puppet-lint/issues/904)
|
30
|
+
- 2.4.x regression with puppet-lint-legacy\_facts-check [\#902](https://github.com/rodjek/puppet-lint/issues/902)
|
31
|
+
- puppet-lint a puppet type in voxpupuli-corosync [\#857](https://github.com/rodjek/puppet-lint/issues/857)
|
32
|
+
- Create and publish a docker image for puppet-lint [\#794](https://github.com/rodjek/puppet-lint/issues/794)
|
33
|
+
|
34
|
+
**Merged pull requests:**
|
35
|
+
|
36
|
+
- Select parameter tokens based on preceeding code token [\#934](https://github.com/rodjek/puppet-lint/pull/934) ([rodjek](https://github.com/rodjek))
|
37
|
+
- Move most CI to GH actions [\#933](https://github.com/rodjek/puppet-lint/pull/933) ([rodjek](https://github.com/rodjek))
|
38
|
+
- Correctly lex non-keyword type tokens in interpolation [\#932](https://github.com/rodjek/puppet-lint/pull/932) ([rodjek](https://github.com/rodjek))
|
39
|
+
- Disallow empty lines between docs and class/define [\#931](https://github.com/rodjek/puppet-lint/pull/931) ([ekohl](https://github.com/ekohl))
|
40
|
+
- Improve Rake task [\#919](https://github.com/rodjek/puppet-lint/pull/919) ([raphink](https://github.com/raphink))
|
41
|
+
- \(\#912\) Count consumed chars not bytes when slurping strings [\#915](https://github.com/rodjek/puppet-lint/pull/915) ([rodjek](https://github.com/rodjek))
|
42
|
+
- Do not rely on tokens array positioning when fixing ensure\_first\_param problems [\#914](https://github.com/rodjek/puppet-lint/pull/914) ([rodjek](https://github.com/rodjek))
|
43
|
+
- Only run test coverage once [\#913](https://github.com/rodjek/puppet-lint/pull/913) ([rodjek](https://github.com/rodjek))
|
44
|
+
- add MIT license to gemspec [\#910](https://github.com/rodjek/puppet-lint/pull/910) ([bastelfreak](https://github.com/bastelfreak))
|
45
|
+
- Improve spelling for some tests [\#909](https://github.com/rodjek/puppet-lint/pull/909) ([usev6](https://github.com/usev6))
|
46
|
+
- \(\#851\) Ensure heredoc ending tag is on its own line [\#903](https://github.com/rodjek/puppet-lint/pull/903) ([rodjek](https://github.com/rodjek))
|
47
|
+
- \(\#794\) Dockerfile for puppet-lint [\#901](https://github.com/rodjek/puppet-lint/pull/901) ([rodjek](https://github.com/rodjek))
|
48
|
+
|
3
49
|
## [2.4.2](https://github.com/rodjek/puppet-lint/tree/2.4.2) (2019-10-31)
|
4
50
|
|
5
51
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.4.1...2.4.2)
|
6
52
|
|
7
53
|
**Fixed bugs:**
|
8
54
|
|
9
|
-
- `ERROR: Syntax error
|
55
|
+
- `ERROR: Syntax error (unterminated string)` with 2.4.1 [\#899](https://github.com/rodjek/puppet-lint/issues/899)
|
10
56
|
- Double quoted string containing variable named "node" mis-tokenized [\#897](https://github.com/rodjek/puppet-lint/issues/897)
|
11
57
|
- 2.4.1 says Syntax error, but code runs fine [\#891](https://github.com/rodjek/puppet-lint/issues/891)
|
12
58
|
- Error with regex variable with multiple `|`s. [\#859](https://github.com/rodjek/puppet-lint/issues/859)
|
@@ -28,8 +74,8 @@
|
|
28
74
|
|
29
75
|
**Fixed bugs:**
|
30
76
|
|
31
|
-
- Puppet-lint 2.4.0 - ERROR: Syntax error on line x [\#887](https://github.com/rodjek/puppet-lint/issues/887)
|
32
77
|
- Puppet-lint 2.4.0 throws misleading warning on double-quoted strings with escaped variables [\#886](https://github.com/rodjek/puppet-lint/issues/886)
|
78
|
+
- Puppet-lint 2.4.0 - ERROR: Syntax error on line x [\#887](https://github.com/rodjek/puppet-lint/issues/887)
|
33
79
|
- Breaks after 2.4.0 upgrade [\#885](https://github.com/rodjek/puppet-lint/issues/885)
|
34
80
|
|
35
81
|
**Merged pull requests:**
|
@@ -43,9 +89,9 @@
|
|
43
89
|
|
44
90
|
**Fixed bugs:**
|
45
91
|
|
92
|
+
- Syntax error on Pattern data type [\#833](https://github.com/rodjek/puppet-lint/issues/833)
|
46
93
|
- Command line options do not override options from config files [\#879](https://github.com/rodjek/puppet-lint/issues/879)
|
47
94
|
- Fix for variables\_not\_enclosed incorrectly handles variables followed by a dash [\#836](https://github.com/rodjek/puppet-lint/issues/836)
|
48
|
-
- Syntax error on Pattern data type [\#833](https://github.com/rodjek/puppet-lint/issues/833)
|
49
95
|
- Error with puppet-lint --fix: NoMethodError: undefined method `next\_token' for nil:NilClass [\#831](https://github.com/rodjek/puppet-lint/issues/831)
|
50
96
|
- TypeError: no implicit conversion of nil into String [\#830](https://github.com/rodjek/puppet-lint/issues/830)
|
51
97
|
- Selector with 'default' case disables check for missing default in outer case statement [\#829](https://github.com/rodjek/puppet-lint/issues/829)
|
@@ -72,14 +118,6 @@
|
|
72
118
|
|
73
119
|
**Merged pull requests:**
|
74
120
|
|
75
|
-
- Handle unenclosed variables followed by dashes when fixing [\#881](https://github.com/rodjek/puppet-lint/pull/881) ([rodjek](https://github.com/rodjek))
|
76
|
-
- Let command line args override config from files [\#880](https://github.com/rodjek/puppet-lint/pull/880) ([usev6](https://github.com/usev6))
|
77
|
-
- Ignore hash keys when checking resource parameter order [\#877](https://github.com/rodjek/puppet-lint/pull/877) ([rodjek](https://github.com/rodjek))
|
78
|
-
- Only look for 'default' at first level of 'case' statement [\#876](https://github.com/rodjek/puppet-lint/pull/876) ([usev6](https://github.com/usev6))
|
79
|
-
- Report syntax error on unbalanced braces [\#875](https://github.com/rodjek/puppet-lint/pull/875) ([rodjek](https://github.com/rodjek))
|
80
|
-
- Include hash/array references when enclosing variables [\#874](https://github.com/rodjek/puppet-lint/pull/874) ([rodjek](https://github.com/rodjek))
|
81
|
-
- Disable quoted\_booleans check by default [\#873](https://github.com/rodjek/puppet-lint/pull/873) ([rodjek](https://github.com/rodjek))
|
82
|
-
- Test against Ruby 2.5 & 2.6 [\#872](https://github.com/rodjek/puppet-lint/pull/872) ([rodjek](https://github.com/rodjek))
|
83
121
|
- README - Add GitHub Actions action [\#868](https://github.com/rodjek/puppet-lint/pull/868) ([ScottBrenner](https://github.com/ScottBrenner))
|
84
122
|
- Update TravisCI config to use trusty image [\#867](https://github.com/rodjek/puppet-lint/pull/867) ([rodjek](https://github.com/rodjek))
|
85
123
|
- Use the default travis rubygems & bundler [\#860](https://github.com/rodjek/puppet-lint/pull/860) ([rodjek](https://github.com/rodjek))
|
@@ -90,6 +128,14 @@
|
|
90
128
|
- Rewrite double quoted string handling for nested interpolation [\#846](https://github.com/rodjek/puppet-lint/pull/846) ([rodjek](https://github.com/rodjek))
|
91
129
|
- Allow for spaces in the heredoc tag [\#841](https://github.com/rodjek/puppet-lint/pull/841) ([jarretlavallee](https://github.com/jarretlavallee))
|
92
130
|
- Recognizes multiline regexes [\#835](https://github.com/rodjek/puppet-lint/pull/835) ([jcbollinger](https://github.com/jcbollinger))
|
131
|
+
- Handle unenclosed variables followed by dashes when fixing [\#881](https://github.com/rodjek/puppet-lint/pull/881) ([rodjek](https://github.com/rodjek))
|
132
|
+
- Let command line args override config from files [\#880](https://github.com/rodjek/puppet-lint/pull/880) ([usev6](https://github.com/usev6))
|
133
|
+
- Ignore hash keys when checking resource parameter order [\#877](https://github.com/rodjek/puppet-lint/pull/877) ([rodjek](https://github.com/rodjek))
|
134
|
+
- Only look for 'default' at first level of 'case' statement [\#876](https://github.com/rodjek/puppet-lint/pull/876) ([usev6](https://github.com/usev6))
|
135
|
+
- Report syntax error on unbalanced braces [\#875](https://github.com/rodjek/puppet-lint/pull/875) ([rodjek](https://github.com/rodjek))
|
136
|
+
- Include hash/array references when enclosing variables [\#874](https://github.com/rodjek/puppet-lint/pull/874) ([rodjek](https://github.com/rodjek))
|
137
|
+
- Disable quoted\_booleans check by default [\#873](https://github.com/rodjek/puppet-lint/pull/873) ([rodjek](https://github.com/rodjek))
|
138
|
+
- Test against Ruby 2.5 & 2.6 [\#872](https://github.com/rodjek/puppet-lint/pull/872) ([rodjek](https://github.com/rodjek))
|
93
139
|
|
94
140
|
## [2.3.6](https://github.com/rodjek/puppet-lint/tree/2.3.6) (2018-07-09)
|
95
141
|
|
@@ -157,11 +203,11 @@
|
|
157
203
|
|
158
204
|
- 2.3.2 - Rakefile's ignore\_paths not respected [\#760](https://github.com/rodjek/puppet-lint/issues/760)
|
159
205
|
- 2.3.1 - Puppet lint fails with 1.8.7-p371 and Puppet 3.8.0 [\#759](https://github.com/rodjek/puppet-lint/issues/759)
|
160
|
-
- 2.3.1: puppet-lint does not show errors/warnings --error-level option. [\#756](https://github.com/rodjek/puppet-lint/issues/756)
|
161
206
|
|
162
207
|
**Merged pull requests:**
|
163
208
|
|
164
209
|
- Add some basic acceptance tests [\#764](https://github.com/rodjek/puppet-lint/pull/764) ([rodjek](https://github.com/rodjek))
|
210
|
+
- Change acceptance tests to work with Ruby 1.8.7 [\#767](https://github.com/rodjek/puppet-lint/pull/767) ([rodjek](https://github.com/rodjek))
|
165
211
|
- Restore Ruby 1.8.7 support [\#763](https://github.com/rodjek/puppet-lint/pull/763) ([rodjek](https://github.com/rodjek))
|
166
212
|
- Don't override ignore\_paths set in rake task with default value [\#762](https://github.com/rodjek/puppet-lint/pull/762) ([rodjek](https://github.com/rodjek))
|
167
213
|
- Add spec for issue raised in \#754 \#756 [\#761](https://github.com/rodjek/puppet-lint/pull/761) ([rodjek](https://github.com/rodjek))
|
@@ -171,9 +217,14 @@
|
|
171
217
|
|
172
218
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.3.1...2.3.2)
|
173
219
|
|
174
|
-
**
|
220
|
+
**Closed issues:**
|
175
221
|
|
176
|
-
-
|
222
|
+
- rake task now failing [\#754](https://github.com/rodjek/puppet-lint/issues/754)
|
223
|
+
|
224
|
+
**Merged pull requests:**
|
225
|
+
|
226
|
+
- 2.3.2 release [\#758](https://github.com/rodjek/puppet-lint/pull/758) ([tphoney](https://github.com/tphoney))
|
227
|
+
- fix logic in method\_missing [\#755](https://github.com/rodjek/puppet-lint/pull/755) ([tphoney](https://github.com/tphoney))
|
177
228
|
|
178
229
|
## [2.3.1](https://github.com/rodjek/puppet-lint/tree/2.3.1) (2017-09-27)
|
179
230
|
|
@@ -181,6 +232,8 @@
|
|
181
232
|
|
182
233
|
**Fixed bugs:**
|
183
234
|
|
235
|
+
- `incompatible encoding regexp match` for non-printing characters in place of space [\#693](https://github.com/rodjek/puppet-lint/issues/693)
|
236
|
+
- Unhandled error case [\#691](https://github.com/rodjek/puppet-lint/issues/691)
|
184
237
|
- NoMethodError: undefined method `type' for nil:NilClass [\#732](https://github.com/rodjek/puppet-lint/issues/732)
|
185
238
|
- NoMethodError: undefined method `end\_with?' for nil:NilClass [\#727](https://github.com/rodjek/puppet-lint/issues/727)
|
186
239
|
- puppet-lint not applying some lint:ignore statements when there are more then 2 on the same line [\#726](https://github.com/rodjek/puppet-lint/issues/726)
|
@@ -188,32 +241,29 @@
|
|
188
241
|
- invalid byte sequence in UTF-8 in selmodule-example.pp [\#714](https://github.com/rodjek/puppet-lint/issues/714)
|
189
242
|
- puppet-lint --fix encountered an error that it doesn't know how to handle [\#706](https://github.com/rodjek/puppet-lint/issues/706)
|
190
243
|
- Mangled file after running puppet-lint due to chained function call [\#703](https://github.com/rodjek/puppet-lint/issues/703)
|
191
|
-
- `incompatible encoding regexp match` for non-printing characters in place of space [\#693](https://github.com/rodjek/puppet-lint/issues/693)
|
192
|
-
- Unhandled error case [\#691](https://github.com/rodjek/puppet-lint/issues/691)
|
193
244
|
|
194
245
|
**Closed issues:**
|
195
246
|
|
196
247
|
- puppet-lint has encountered an error that it doesn't know how to handle [\#750](https://github.com/rodjek/puppet-lint/issues/750)
|
197
|
-
- Variable use like "a+1 = ${$a + 1}" isn't reported but silently changed by --fix to "a+1 = ${a} + 1" [\#749](https://github.com/rodjek/puppet-lint/issues/749)
|
198
|
-
- using --fix changes line endings [\#748](https://github.com/rodjek/puppet-lint/issues/748)
|
199
248
|
- Puppet lint syntax error - puppet parser validate no issues [\#746](https://github.com/rodjek/puppet-lint/issues/746)
|
200
|
-
- Error not handled [\#745](https://github.com/rodjek/puppet-lint/issues/745)
|
201
249
|
- Whoops, not sure why. ArgumentError: bad value for range [\#742](https://github.com/rodjek/puppet-lint/issues/742)
|
202
250
|
- ArgumentError: bad value for range [\#741](https://github.com/rodjek/puppet-lint/issues/741)
|
203
|
-
- Line numbers off after multi-line strings with variables [\#736](https://github.com/rodjek/puppet-lint/issues/736)
|
204
251
|
- Whoops! It looks like puppet-lint has encountered an error [\#729](https://github.com/rodjek/puppet-lint/issues/729)
|
205
252
|
- puppet lint config log\_format not working [\#725](https://github.com/rodjek/puppet-lint/issues/725)
|
206
|
-
- Quoted booleans in Puppet5 for Enum type declarations. [\#720](https://github.com/rodjek/puppet-lint/issues/720)
|
207
253
|
- Chaining arrow syntax fix introduces trailing whitespaces [\#695](https://github.com/rodjek/puppet-lint/issues/695)
|
208
254
|
- Refactor check\_comments.rb [\#587](https://github.com/rodjek/puppet-lint/issues/587)
|
209
255
|
- Puppethack 12/2016 Issues [\#583](https://github.com/rodjek/puppet-lint/issues/583)
|
210
256
|
- Allow multiple block-level ignore comments [\#498](https://github.com/rodjek/puppet-lint/issues/498)
|
211
257
|
- puppet-lint crashes with "invalid byte sequence in UTF-8 \(ArgumentError\)" [\#458](https://github.com/rodjek/puppet-lint/issues/458)
|
258
|
+
- Variable use like "a+1 = ${$a + 1}" isn't reported but silently changed by --fix to "a+1 = ${a} + 1" [\#749](https://github.com/rodjek/puppet-lint/issues/749)
|
259
|
+
- using --fix changes line endings [\#748](https://github.com/rodjek/puppet-lint/issues/748)
|
260
|
+
- Error not handled [\#745](https://github.com/rodjek/puppet-lint/issues/745)
|
261
|
+
- Line numbers off after multi-line strings with variables [\#736](https://github.com/rodjek/puppet-lint/issues/736)
|
262
|
+
- Quoted booleans in Puppet5 for Enum type declarations. [\#720](https://github.com/rodjek/puppet-lint/issues/720)
|
263
|
+
- Plugins reorganization: One file per check [\#657](https://github.com/rodjek/puppet-lint/issues/657)
|
212
264
|
|
213
265
|
**Merged pull requests:**
|
214
266
|
|
215
|
-
- Render the ${} enclosures as part of the string tokens [\#752](https://github.com/rodjek/puppet-lint/pull/752) ([rodjek](https://github.com/rodjek))
|
216
|
-
- Open manifest as binary when writing fixed manifest [\#751](https://github.com/rodjek/puppet-lint/pull/751) ([rodjek](https://github.com/rodjek))
|
217
267
|
- Remove monkeypatches to implement String\#% [\#744](https://github.com/rodjek/puppet-lint/pull/744) ([rodjek](https://github.com/rodjek))
|
218
268
|
- Add unit tests for PuppetLint::Checks [\#743](https://github.com/rodjek/puppet-lint/pull/743) ([rodjek](https://github.com/rodjek))
|
219
269
|
- Update CI configuration [\#739](https://github.com/rodjek/puppet-lint/pull/739) ([rodjek](https://github.com/rodjek))
|
@@ -222,13 +272,17 @@
|
|
222
272
|
- Split control comments into words before parsing [\#735](https://github.com/rodjek/puppet-lint/pull/735) ([rodjek](https://github.com/rodjek))
|
223
273
|
- Handle unicode spaces in the tokeniser [\#734](https://github.com/rodjek/puppet-lint/pull/734) ([rodjek](https://github.com/rodjek))
|
224
274
|
- Handle SE Linux policy package files [\#733](https://github.com/rodjek/puppet-lint/pull/733) ([rodjek](https://github.com/rodjek))
|
275
|
+
- Chaining arrow syntax fix introduces trailing whitespaces [\#708](https://github.com/rodjek/puppet-lint/pull/708) ([rnelson0](https://github.com/rnelson0))
|
276
|
+
- Plugins: Improve code readability [\#658](https://github.com/rodjek/puppet-lint/pull/658) ([Darhazer](https://github.com/Darhazer))
|
277
|
+
- Render the ${} enclosures as part of the string tokens [\#752](https://github.com/rodjek/puppet-lint/pull/752) ([rodjek](https://github.com/rodjek))
|
278
|
+
- Open manifest as binary when writing fixed manifest [\#751](https://github.com/rodjek/puppet-lint/pull/751) ([rodjek](https://github.com/rodjek))
|
225
279
|
- Take into account Optional data type when checking parameter order [\#731](https://github.com/rodjek/puppet-lint/pull/731) ([rodjek](https://github.com/rodjek))
|
226
280
|
- Read the manifest files as UTF-8 [\#730](https://github.com/rodjek/puppet-lint/pull/730) ([rodjek](https://github.com/rodjek))
|
227
281
|
- Improve handling of unterminated double quoted strings [\#728](https://github.com/rodjek/puppet-lint/pull/728) ([rodjek](https://github.com/rodjek))
|
228
|
-
- Chaining arrow syntax fix introduces trailing whitespaces [\#708](https://github.com/rodjek/puppet-lint/pull/708) ([rnelson0](https://github.com/rnelson0))
|
229
282
|
- Add helper methods to add and remove tokens while maintaining the token links [\#694](https://github.com/rodjek/puppet-lint/pull/694) ([Darhazer](https://github.com/Darhazer))
|
283
|
+
- Check that arrow is on the line of right operand [\#672](https://github.com/rodjek/puppet-lint/pull/672) ([Darhazer](https://github.com/Darhazer))
|
284
|
+
- One file per plugin. Fixes \#657 [\#671](https://github.com/rodjek/puppet-lint/pull/671) ([Darhazer](https://github.com/Darhazer))
|
230
285
|
- Code style improvements [\#661](https://github.com/rodjek/puppet-lint/pull/661) ([Darhazer](https://github.com/Darhazer))
|
231
|
-
- Plugins: Improve code readability [\#658](https://github.com/rodjek/puppet-lint/pull/658) ([Darhazer](https://github.com/Darhazer))
|
232
286
|
|
233
287
|
## [2.3.0](https://github.com/rodjek/puppet-lint/tree/2.3.0) (2017-07-12)
|
234
288
|
|
@@ -238,21 +292,18 @@
|
|
238
292
|
|
239
293
|
- Inappropriate =\> Indentation Warning in Hash [\#698](https://github.com/rodjek/puppet-lint/issues/698)
|
240
294
|
- Regression: arrow\_alignment check broken between 2.1.1 and 2.2.x releases when left side contains text after variable interpolation [\#697](https://github.com/rodjek/puppet-lint/issues/697)
|
241
|
-
- Check `
|
295
|
+
- Check `arrow_on_right_operand_line` is undocumented. [\#688](https://github.com/rodjek/puppet-lint/issues/688)
|
242
296
|
|
243
297
|
**Merged pull requests:**
|
244
298
|
|
245
299
|
- \(maint\) Fixes puppet-lint json formatting to output valid json to stdout [\#719](https://github.com/rodjek/puppet-lint/pull/719) ([bmjen](https://github.com/bmjen))
|
246
300
|
- Permit puppet-lint to load "prerelease" gems [\#718](https://github.com/rodjek/puppet-lint/pull/718) ([kpaulisse](https://github.com/kpaulisse))
|
247
301
|
- Document the configuration file better [\#713](https://github.com/rodjek/puppet-lint/pull/713) ([binford2k](https://github.com/binford2k))
|
248
|
-
- Replace deprecated plugin with an improved alternative. [\#712](https://github.com/rodjek/puppet-lint/pull/712) ([deanwilson](https://github.com/deanwilson))
|
249
302
|
- Readme formatting fixes [\#709](https://github.com/rodjek/puppet-lint/pull/709) ([dbeckham](https://github.com/dbeckham))
|
250
303
|
- Readme edit [\#707](https://github.com/rodjek/puppet-lint/pull/707) ([jbondpdx](https://github.com/jbondpdx))
|
251
304
|
- Take into account length of DQPOST token when updating column number [\#701](https://github.com/rodjek/puppet-lint/pull/701) ([dbeckham](https://github.com/dbeckham))
|
252
305
|
- Add pattern support to rake task [\#700](https://github.com/rodjek/puppet-lint/pull/700) ([dbeckham](https://github.com/dbeckham))
|
253
306
|
- Note arrow\_on\_right\_operand\_line in the README [\#690](https://github.com/rodjek/puppet-lint/pull/690) ([rodjek](https://github.com/rodjek))
|
254
|
-
- Document the arrow\_on\_right\_operand\_line check on the website [\#689](https://github.com/rodjek/puppet-lint/pull/689) ([rodjek](https://github.com/rodjek))
|
255
|
-
- One file per plugin. Fixes \#657 [\#671](https://github.com/rodjek/puppet-lint/pull/671) ([Darhazer](https://github.com/Darhazer))
|
256
307
|
|
257
308
|
## [2.2.1](https://github.com/rodjek/puppet-lint/tree/2.2.1) (2017-03-29)
|
258
309
|
|
@@ -276,7 +327,6 @@
|
|
276
327
|
|
277
328
|
**Closed issues:**
|
278
329
|
|
279
|
-
- Plugins reorganization: One file per check [\#657](https://github.com/rodjek/puppet-lint/issues/657)
|
280
330
|
- 2.1.1 git tag [\#652](https://github.com/rodjek/puppet-lint/issues/652)
|
281
331
|
- Quoted boolean triggers on the command 'true' [\#646](https://github.com/rodjek/puppet-lint/issues/646)
|
282
332
|
- Namevars detected as optional parameters [\#633](https://github.com/rodjek/puppet-lint/issues/633)
|
@@ -302,16 +352,14 @@
|
|
302
352
|
- Heredoc triggers exception in arrow\_alignment check [\#578](https://github.com/rodjek/puppet-lint/issues/578)
|
303
353
|
- Each + With = Fake positive top-scope variable detection [\#576](https://github.com/rodjek/puppet-lint/issues/576)
|
304
354
|
- Top-scope variable warning on nested each loops [\#548](https://github.com/rodjek/puppet-lint/issues/548)
|
305
|
-
- `
|
355
|
+
- `arrow_alignment --fix` doesn't indent keys when introducing line breaks; erroneously reports success [\#506](https://github.com/rodjek/puppet-lint/issues/506)
|
306
356
|
- heredoc escape gives syntax error [\#430](https://github.com/rodjek/puppet-lint/issues/430)
|
307
357
|
- NoMethodError when multiple heredocs are used [\#395](https://github.com/rodjek/puppet-lint/issues/395)
|
308
358
|
|
309
359
|
**Merged pull requests:**
|
310
360
|
|
311
361
|
- Support double quoted strings inside interpolated values in double quoted strings [\#676](https://github.com/rodjek/puppet-lint/pull/676) ([rodjek](https://github.com/rodjek))
|
312
|
-
- Check that arrow is on the line of right operand [\#672](https://github.com/rodjek/puppet-lint/pull/672) ([Darhazer](https://github.com/Darhazer))
|
313
362
|
- Deal with ruby 1.8.7 gem issues [\#630](https://github.com/rodjek/puppet-lint/pull/630) ([mterzo](https://github.com/mterzo))
|
314
|
-
- Plugin review; disable unnecessary plugins [\#567](https://github.com/rodjek/puppet-lint/pull/567) ([rnelson0](https://github.com/rnelson0))
|
315
363
|
- Ignore selectors when finding resource type [\#678](https://github.com/rodjek/puppet-lint/pull/678) ([rodjek](https://github.com/rodjek))
|
316
364
|
- Fix for arrow\_alignment bugs in \#506 [\#677](https://github.com/rodjek/puppet-lint/pull/677) ([rodjek](https://github.com/rodjek))
|
317
365
|
- Don't silently remove unnecessary $ from enclosed variables [\#674](https://github.com/rodjek/puppet-lint/pull/674) ([rodjek](https://github.com/rodjek))
|
@@ -362,14 +410,12 @@
|
|
362
410
|
|
363
411
|
- Excessive number of warnings from code\_on\_top\_scope [\#579](https://github.com/rodjek/puppet-lint/issues/579)
|
364
412
|
- Syntax error for references starting with Regexp [\#566](https://github.com/rodjek/puppet-lint/issues/566)
|
365
|
-
- --fix flag discoverability is low [\#556](https://github.com/rodjek/puppet-lint/issues/556)
|
366
413
|
- Module names must only contain lowercase letters [\#554](https://github.com/rodjek/puppet-lint/issues/554)
|
367
414
|
- Remove formatting with `linenumber` [\#539](https://github.com/rodjek/puppet-lint/issues/539)
|
368
415
|
- names\_containing\_dash is broken and unignorable [\#534](https://github.com/rodjek/puppet-lint/issues/534)
|
369
416
|
- Puppet-lint 2.0 fails on unquoted string starting with underscore. [\#531](https://github.com/rodjek/puppet-lint/issues/531)
|
370
417
|
- puppet-lint reports incorrect line numbers [\#403](https://github.com/rodjek/puppet-lint/issues/403)
|
371
418
|
- Release version 2.1.0 [\#610](https://github.com/rodjek/puppet-lint/issues/610)
|
372
|
-
- No documentation on how to use plugins [\#602](https://github.com/rodjek/puppet-lint/issues/602)
|
373
419
|
- incorrect error on inline template [\#545](https://github.com/rodjek/puppet-lint/issues/545)
|
374
420
|
- parameter\_order check does not work default hash is added in parameter [\#544](https://github.com/rodjek/puppet-lint/issues/544)
|
375
421
|
- Incorrect warning of required parameter when using array that includes variable. [\#537](https://github.com/rodjek/puppet-lint/issues/537)
|
@@ -379,7 +425,6 @@
|
|
379
425
|
- Start a CHANGELOG, make updates part of the build workflow [\#479](https://github.com/rodjek/puppet-lint/issues/479)
|
380
426
|
- Nested future scope blocks lose local variables from parent scopes. [\#456](https://github.com/rodjek/puppet-lint/issues/456)
|
381
427
|
- block-local variables \(|$x|\) don't properly get recognized when used as arrays or hashes [\#450](https://github.com/rodjek/puppet-lint/issues/450)
|
382
|
-
- lint ignore comments feature not documented [\#369](https://github.com/rodjek/puppet-lint/issues/369)
|
383
428
|
|
384
429
|
**Merged pull requests:**
|
385
430
|
|
@@ -392,17 +437,13 @@
|
|
392
437
|
- Multi line strings [\#570](https://github.com/rodjek/puppet-lint/pull/570) ([jiuka](https://github.com/jiuka))
|
393
438
|
- @node\_indexes should be reset too. [\#569](https://github.com/rodjek/puppet-lint/pull/569) ([jiuka](https://github.com/jiuka))
|
394
439
|
- True up reserved keywords [\#564](https://github.com/rodjek/puppet-lint/pull/564) ([rnelson0](https://github.com/rnelson0))
|
395
|
-
- \[561\] Provide style guide references for each check [\#562](https://github.com/rodjek/puppet-lint/pull/562) ([rnelson0](https://github.com/rnelson0))
|
396
|
-
- Add names\_containing\_uppercase docs [\#559](https://github.com/rodjek/puppet-lint/pull/559) ([arrdem](https://github.com/arrdem))
|
397
440
|
- Implement a linter for uppercase class names [\#558](https://github.com/rodjek/puppet-lint/pull/558) ([arrdem](https://github.com/arrdem))
|
398
441
|
- Include --fix usage instructions [\#557](https://github.com/rodjek/puppet-lint/pull/557) ([QuinnyPig](https://github.com/QuinnyPig))
|
399
|
-
- Add puppet-lint plugins from deanwilson to plugins page [\#553](https://github.com/rodjek/puppet-lint/pull/553) ([deanwilson](https://github.com/deanwilson))
|
400
442
|
- Updates PR for \#223 [\#552](https://github.com/rodjek/puppet-lint/pull/552) ([binford2k](https://github.com/binford2k))
|
401
|
-
- add legacy\_facts [\#543](https://github.com/rodjek/puppet-lint/pull/543) ([mmckinst](https://github.com/mmckinst))
|
402
443
|
- Remove formatting with `linenumber` [\#540](https://github.com/rodjek/puppet-lint/pull/540) ([rski](https://github.com/rski))
|
403
444
|
- Potential README changes [\#420](https://github.com/rodjek/puppet-lint/pull/420) ([rothsa](https://github.com/rothsa))
|
404
445
|
- Add fix to "ensure found on line but it's not the first attribute" [\#375](https://github.com/rodjek/puppet-lint/pull/375) ([sathieu](https://github.com/sathieu))
|
405
|
-
-
|
446
|
+
- Checks for code outside class/define block [\#223](https://github.com/rodjek/puppet-lint/pull/223) ([dLobatog](https://github.com/dLobatog))
|
406
447
|
- \(\#369\) Document existence of control comments [\#600](https://github.com/rodjek/puppet-lint/pull/600) ([rnelson0](https://github.com/rnelson0))
|
407
448
|
- \(\#517\) Update the allowed tokens prior to a regex [\#594](https://github.com/rodjek/puppet-lint/pull/594) ([rnelson0](https://github.com/rnelson0))
|
408
449
|
- \(\#531\) Treat barewords beginning with an underscore as :NAME tokens [\#593](https://github.com/rodjek/puppet-lint/pull/593) ([rnelson0](https://github.com/rnelson0))
|
@@ -416,7 +457,7 @@
|
|
416
457
|
|
417
458
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.0.1...2.0.2)
|
418
459
|
|
419
|
-
**
|
460
|
+
**Implemented enhancements:**
|
420
461
|
|
421
462
|
- Option to choose version of the puppet style guide [\#190](https://github.com/rodjek/puppet-lint/issues/190)
|
422
463
|
|
@@ -428,6 +469,10 @@
|
|
428
469
|
|
429
470
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.0.0...2.0.1)
|
430
471
|
|
472
|
+
**Implemented enhancements:**
|
473
|
+
|
474
|
+
- Allow specifying the default enable status of a check [\#484](https://github.com/rodjek/puppet-lint/issues/484)
|
475
|
+
|
431
476
|
**Closed issues:**
|
432
477
|
|
433
478
|
- Changelog on Github pages is out of date [\#520](https://github.com/rodjek/puppet-lint/issues/520)
|
@@ -437,7 +482,6 @@
|
|
437
482
|
- question: 1.1.0 --\> 2.0.0 new checks [\#499](https://github.com/rodjek/puppet-lint/issues/499)
|
438
483
|
- \[\#puppethack\] disable\_char\_check doesn't work [\#493](https://github.com/rodjek/puppet-lint/issues/493)
|
439
484
|
- Error fixing indent in possion 0 [\#489](https://github.com/rodjek/puppet-lint/issues/489)
|
440
|
-
- Allow specifying the default enable status of a check [\#484](https://github.com/rodjek/puppet-lint/issues/484)
|
441
485
|
- Empty lines with trailing whitespace triggers redundant soft tabs error. [\#478](https://github.com/rodjek/puppet-lint/issues/478)
|
442
486
|
- Little problem with =\> when variables are used as key [\#472](https://github.com/rodjek/puppet-lint/issues/472)
|
443
487
|
- Question: using config file, with custom path [\#466](https://github.com/rodjek/puppet-lint/issues/466)
|
@@ -458,7 +502,6 @@
|
|
458
502
|
- puppet-lint crash [\#422](https://github.com/rodjek/puppet-lint/issues/422)
|
459
503
|
- PuppetLint.configuration.ignore\_paths ignored by puppet-lint [\#417](https://github.com/rodjek/puppet-lint/issues/417)
|
460
504
|
- Quoted boolean check should only check values [\#415](https://github.com/rodjek/puppet-lint/issues/415)
|
461
|
-
- nested ensure misdetected as not coming first. [\#410](https://github.com/rodjek/puppet-lint/issues/410)
|
462
505
|
- tabs before code issue [\#402](https://github.com/rodjek/puppet-lint/issues/402)
|
463
506
|
- Disabled checks aren't actually disabled, output is merely ignored. [\#400](https://github.com/rodjek/puppet-lint/issues/400)
|
464
507
|
- file modes doesn't have to be 4 digit octal [\#394](https://github.com/rodjek/puppet-lint/issues/394)
|
@@ -481,13 +524,8 @@
|
|
481
524
|
|
482
525
|
**Merged pull requests:**
|
483
526
|
|
484
|
-
- Continue supporting Ruby \< 2 [\#529](https://github.com/rodjek/puppet-lint/pull/529) ([ghoneycutt](https://github.com/ghoneycutt))
|
485
|
-
- Use underscores instead of CamelCase in example [\#527](https://github.com/rodjek/puppet-lint/pull/527) ([ghoneycutt](https://github.com/ghoneycutt))
|
486
|
-
- new linter for top\_scope\_facts [\#526](https://github.com/rodjek/puppet-lint/pull/526) ([mmckinst](https://github.com/mmckinst))
|
487
527
|
- \(GH462\) Multi-line comments can now be fixed. [\#525](https://github.com/rodjek/puppet-lint/pull/525) ([rnelson0](https://github.com/rnelson0))
|
488
528
|
- New additions of protected variables [\#524](https://github.com/rodjek/puppet-lint/pull/524) ([rnelson0](https://github.com/rnelson0))
|
489
|
-
- Document new checks in puppet-lint 2.0.0+ [\#523](https://github.com/rodjek/puppet-lint/pull/523) ([rnelson0](https://github.com/rnelson0))
|
490
|
-
- Remove github pages changelog [\#522](https://github.com/rodjek/puppet-lint/pull/522) ([3flex](https://github.com/3flex))
|
491
529
|
- \(GH366\) Arrow Alignment fix crashes with tabs [\#515](https://github.com/rodjek/puppet-lint/pull/515) ([rnelson0](https://github.com/rnelson0))
|
492
530
|
- Make params disabled [\#511](https://github.com/rodjek/puppet-lint/pull/511) ([binford2k](https://github.com/binford2k))
|
493
531
|
- Removing params class check [\#510](https://github.com/rodjek/puppet-lint/pull/510) ([cvquesty](https://github.com/cvquesty))
|
@@ -497,7 +535,6 @@
|
|
497
535
|
- This adds a disabled-by-default 80chars check [\#495](https://github.com/rodjek/puppet-lint/pull/495) ([binford2k](https://github.com/binford2k))
|
498
536
|
- Add better description of the problem in arrow\_alignment check [\#492](https://github.com/rodjek/puppet-lint/pull/492) ([rnelson0](https://github.com/rnelson0))
|
499
537
|
- \(GH410\) Limit ensure\_first\_param check to certain resources [\#490](https://github.com/rodjek/puppet-lint/pull/490) ([rnelson0](https://github.com/rnelson0))
|
500
|
-
- add lint test for resource references [\#486](https://github.com/rodjek/puppet-lint/pull/486) ([tuxmea](https://github.com/tuxmea))
|
501
538
|
- Regression from 418: duplicate constant [\#483](https://github.com/rodjek/puppet-lint/pull/483) ([rnelson0](https://github.com/rnelson0))
|
502
539
|
- Add fully-detailed CHANGELOG [\#481](https://github.com/rodjek/puppet-lint/pull/481) ([petems](https://github.com/petems))
|
503
540
|
- Add a CHANGELOG [\#480](https://github.com/rodjek/puppet-lint/pull/480) ([rnelson0](https://github.com/rnelson0))
|
@@ -511,26 +548,8 @@
|
|
511
548
|
- Check that variables are lowercase [\#418](https://github.com/rodjek/puppet-lint/pull/418) ([rothsa](https://github.com/rothsa))
|
512
549
|
|
513
550
|
## [2.0.0](https://github.com/rodjek/puppet-lint/tree/2.0.0) (2016-06-22)
|
514
|
-
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/1.1.0...2.0.0)
|
515
|
-
|
516
|
-
puppet-lint 2.0.0 is a breaking change. Specifically, the renaming of the line length test was changed from `80chars` to `140chars`. You may need to adjust your configuration and lint checks. For example:
|
517
|
-
```ruby
|
518
|
-
# Line length test is 80 chars in puppet-lint 1.1.0
|
519
|
-
PuppetLint.configuration.send('disable_80chars')
|
520
|
-
# Line length test is 140 chars in puppet-lint 2.x
|
521
|
-
PuppetLint.configuration.send('disable_140chars')
|
522
|
-
```
|
523
|
-
|
524
|
-
You may also need to adjust your Gemfile if you are pointing directly at git:
|
525
|
-
```ruby
|
526
|
-
# old
|
527
|
-
gem 'puppet-lint', :require => false, :git => 'https://github.com/rodjek/puppet-lint.git'
|
528
551
|
|
529
|
-
|
530
|
-
gem 'puppet-lint', '~> 2.0'
|
531
|
-
```
|
532
|
-
|
533
|
-
If the additional gems you use for checks are pinned to 1.x, you should pin puppet-lint to `'~> 1.0'` or `'>= 1.0', '< 3.0'` until updated check gems are released.
|
552
|
+
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/1.1.0...2.0.0)
|
534
553
|
|
535
554
|
**Closed issues:**
|
536
555
|
|
@@ -577,8 +596,6 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
577
596
|
- \(GH443\) Release 2.0.0 PR [\#477](https://github.com/rodjek/puppet-lint/pull/477) ([rnelson0](https://github.com/rnelson0))
|
578
597
|
- Fix arrow aligment check in multiple resources declaration [\#476](https://github.com/rodjek/puppet-lint/pull/476) ([wybczu](https://github.com/wybczu))
|
579
598
|
- Fix issue \#450: block-local variables aren't recognized with subscripts [\#453](https://github.com/rodjek/puppet-lint/pull/453) ([jearls](https://github.com/jearls))
|
580
|
-
- Adding package\_ensure plugin [\#448](https://github.com/rodjek/puppet-lint/pull/448) ([danzilio](https://github.com/danzilio))
|
581
|
-
- Update documentation for 140chars [\#440](https://github.com/rodjek/puppet-lint/pull/440) ([keeleysam](https://github.com/keeleysam))
|
582
599
|
- Changed character width to 140. [\#419](https://github.com/rodjek/puppet-lint/pull/419) ([potto007](https://github.com/potto007))
|
583
600
|
- Fix arrow\_alignment check to not raise exception when line isn't indented [\#413](https://github.com/rodjek/puppet-lint/pull/413) ([rodjek](https://github.com/rodjek))
|
584
601
|
- Fix puppet:// url check to catch double quoted strings [\#407](https://github.com/rodjek/puppet-lint/pull/407) ([paulgeringer](https://github.com/paulgeringer))
|
@@ -591,24 +608,17 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
591
608
|
- Detect trailing whitespace on lines with no code [\#386](https://github.com/rodjek/puppet-lint/pull/386) ([rodjek](https://github.com/rodjek))
|
592
609
|
- Save the raw value of MLCOMMENT tokens to use when rendering back to a manifest [\#385](https://github.com/rodjek/puppet-lint/pull/385) ([rodjek](https://github.com/rodjek))
|
593
610
|
- Don't suppress nil values in manifest\_lines [\#384](https://github.com/rodjek/puppet-lint/pull/384) ([rodjek](https://github.com/rodjek))
|
594
|
-
- Update index.md [\#377](https://github.com/rodjek/puppet-lint/pull/377) ([mcanevet](https://github.com/mcanevet))
|
595
611
|
- Only clear task if it's already defined [\#376](https://github.com/rodjek/puppet-lint/pull/376) ([domcleal](https://github.com/domcleal))
|
596
|
-
- add strict\_indent check to community plugins [\#371](https://github.com/rodjek/puppet-lint/pull/371) ([relud](https://github.com/relud))
|
597
612
|
- Nested cases [\#368](https://github.com/rodjek/puppet-lint/pull/368) ([jonnangle](https://github.com/jonnangle))
|
598
613
|
- rpearce: Allow the use of facts\[\] and trusted\[\] as per Puppet 3.5+ [\#362](https://github.com/rodjek/puppet-lint/pull/362) ([rjpearce](https://github.com/rjpearce))
|
599
|
-
- plugins: Add absolute template path check [\#353](https://github.com/rodjek/puppet-lint/pull/353) ([3flex](https://github.com/3flex))
|
600
|
-
- Update index.md [\#352](https://github.com/rodjek/puppet-lint/pull/352) ([mcanevet](https://github.com/mcanevet))
|
601
614
|
- Add node\_indexes method [\#351](https://github.com/rodjek/puppet-lint/pull/351) ([mcanevet](https://github.com/mcanevet))
|
602
615
|
- Don't attempt to fix ignored problems [\#349](https://github.com/rodjek/puppet-lint/pull/349) ([rodjek](https://github.com/rodjek))
|
603
616
|
- Handle case where a colon is the last token in a file [\#346](https://github.com/rodjek/puppet-lint/pull/346) ([rodjek](https://github.com/rodjek))
|
604
617
|
- Fix bug in unquoted\_node\_name to support multiple node blocks [\#345](https://github.com/rodjek/puppet-lint/pull/345) ([rodjek](https://github.com/rodjek))
|
605
618
|
- Catch Errno::EACCES when reading a puppet-lint.rc out of HOME [\#342](https://github.com/rodjek/puppet-lint/pull/342) ([rodjek](https://github.com/rodjek))
|
606
|
-
- Generate line numbers for the plugin tutorial code examples [\#340](https://github.com/rodjek/puppet-lint/pull/340) ([rodjek](https://github.com/rodjek))
|
607
619
|
- Add support for '--relative' option in new Rake::Task format. [\#334](https://github.com/rodjek/puppet-lint/pull/334) ([fatmcgav](https://github.com/fatmcgav))
|
608
620
|
- fix \#331 - clear any pre-\(auto-\)existing tasks [\#332](https://github.com/rodjek/puppet-lint/pull/332) ([duritong](https://github.com/duritong))
|
609
621
|
- Don't warn for arrow alignment for single-element hashes [\#330](https://github.com/rodjek/puppet-lint/pull/330) ([domcleal](https://github.com/domcleal))
|
610
|
-
- Document multiple commands in a single control comment [\#329](https://github.com/rodjek/puppet-lint/pull/329) ([domcleal](https://github.com/domcleal))
|
611
|
-
- Add parameter\_documentation/param-docs plugin [\#328](https://github.com/rodjek/puppet-lint/pull/328) ([domcleal](https://github.com/domcleal))
|
612
622
|
- Alternative to \#289: :error on either class names and defines [\#290](https://github.com/rodjek/puppet-lint/pull/290) ([ppp0](https://github.com/ppp0))
|
613
623
|
|
614
624
|
## [1.1.0](https://github.com/rodjek/puppet-lint/tree/1.1.0) (2014-09-23)
|
@@ -657,14 +667,18 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
657
667
|
**Merged pull requests:**
|
658
668
|
|
659
669
|
- Handle empty blocks in arrow\_alignment [\#302](https://github.com/rodjek/puppet-lint/pull/302) ([rodjek](https://github.com/rodjek))
|
660
|
-
- fix typo in the links to 'trailing\_newline' plugin [\#300](https://github.com/rodjek/puppet-lint/pull/300) ([3flex](https://github.com/3flex))
|
661
|
-
- Comments on developer puppet-lint checks [\#299](https://github.com/rodjek/puppet-lint/pull/299) ([jfryman](https://github.com/jfryman))
|
662
670
|
- \[Fixes \#291\] Ignore index braces for scope variables [\#303](https://github.com/rodjek/puppet-lint/pull/303) ([dcarley](https://github.com/dcarley))
|
663
671
|
|
664
672
|
## [1.0.0](https://github.com/rodjek/puppet-lint/tree/1.0.0) (2014-08-18)
|
665
673
|
|
666
674
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/0.4.0.pre1...1.0.0)
|
667
675
|
|
676
|
+
**Implemented enhancements:**
|
677
|
+
|
678
|
+
- Need a way to ignore a lint check for a particular line [\#247](https://github.com/rodjek/puppet-lint/issues/247)
|
679
|
+
- Trailing line comment to disable check [\#214](https://github.com/rodjek/puppet-lint/issues/214)
|
680
|
+
- disable tests on arbitrary lines or over blocks of code [\#68](https://github.com/rodjek/puppet-lint/issues/68)
|
681
|
+
|
668
682
|
**Closed issues:**
|
669
683
|
|
670
684
|
- Stop complaining about things unsupported versions of Puppet won't support. [\#281](https://github.com/rodjek/puppet-lint/issues/281)
|
@@ -699,15 +713,13 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
699
713
|
- Future parser loop should allow for inline variable declaration without scoping [\#264](https://github.com/rodjek/puppet-lint/issues/264)
|
700
714
|
- Double Quoted Strings - should allow for strings containing single quotes [\#263](https://github.com/rodjek/puppet-lint/issues/263)
|
701
715
|
- puppet-lint should not warn "string containing only a variable" when it's used to create a hash [\#261](https://github.com/rodjek/puppet-lint/issues/261)
|
702
|
-
- ERROR: Syntax error \(try running `puppet parser validate
|
716
|
+
- ERROR: Syntax error \(try running `puppet parser validate <file>`\) on line 15 [\#258](https://github.com/rodjek/puppet-lint/issues/258)
|
703
717
|
- Variables in each incorrectly idenentified as top scope. [\#249](https://github.com/rodjek/puppet-lint/issues/249)
|
704
718
|
- puppet-lint -f makes a mess of double-quoted strings that contain single quotes [\#248](https://github.com/rodjek/puppet-lint/issues/248)
|
705
|
-
- Need a way to ignore a lint check for a particular line [\#247](https://github.com/rodjek/puppet-lint/issues/247)
|
706
719
|
- Puppetlint should ignore template lines \> 80 characters [\#233](https://github.com/rodjek/puppet-lint/issues/233)
|
707
720
|
- Syntax error when parser future features used [\#232](https://github.com/rodjek/puppet-lint/issues/232)
|
708
721
|
- Issue with puppet-lint -f and trailing whitespace [\#224](https://github.com/rodjek/puppet-lint/issues/224)
|
709
722
|
- escape a variable interpolation to skip checking [\#219](https://github.com/rodjek/puppet-lint/issues/219)
|
710
|
-
- Trailing line comment to disable check [\#214](https://github.com/rodjek/puppet-lint/issues/214)
|
711
723
|
- wrong title for \>80char per line check site [\#209](https://github.com/rodjek/puppet-lint/issues/209)
|
712
724
|
- `rake lint` should respect .puppet-lint.rc in root of module [\#202](https://github.com/rodjek/puppet-lint/issues/202)
|
713
725
|
- lint analyzes inlined ruby code [\#201](https://github.com/rodjek/puppet-lint/issues/201)
|
@@ -718,11 +730,9 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
718
730
|
- Puppet lint seems to ignore --no-class\_parameter\_defaults-check when inheriting the params class [\#173](https://github.com/rodjek/puppet-lint/issues/173)
|
719
731
|
- Quoting top level variable in class parameter cause false warning [\#170](https://github.com/rodjek/puppet-lint/issues/170)
|
720
732
|
- exec and Bash vars, false-postive "single quoted string containing a variable found on line" [\#113](https://github.com/rodjek/puppet-lint/issues/113)
|
721
|
-
- disable tests on arbitrary lines or over blocks of code [\#68](https://github.com/rodjek/puppet-lint/issues/68)
|
722
733
|
|
723
734
|
**Merged pull requests:**
|
724
735
|
|
725
|
-
- Checks for code outside class/define block [\#223](https://github.com/rodjek/puppet-lint/pull/223) ([dLobatog](https://github.com/dLobatog))
|
726
736
|
- explains in README that puppet-lint is not for checking syntax [\#186](https://github.com/rodjek/puppet-lint/pull/186) ([ghoneycutt](https://github.com/ghoneycutt))
|
727
737
|
- Use the current workdir as reference to calculate the expanded\_path of a filename [\#175](https://github.com/rodjek/puppet-lint/pull/175) ([vStone](https://github.com/vStone))
|
728
738
|
- Update code documentation [\#298](https://github.com/rodjek/puppet-lint/pull/298) ([rodjek](https://github.com/rodjek))
|
@@ -761,7 +771,6 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
761
771
|
- Bad hash indenting with --fix where members declared on a single line [\#217](https://github.com/rodjek/puppet-lint/pull/217) ([aelse](https://github.com/aelse))
|
762
772
|
- Handle variables with array & hash references [\#212](https://github.com/rodjek/puppet-lint/pull/212) ([rodjek](https://github.com/rodjek))
|
763
773
|
- Add Modulo token [\#207](https://github.com/rodjek/puppet-lint/pull/207) ([dalen](https://github.com/dalen))
|
764
|
-
- Fix class\_inherits\_from\_params\_class docs link [\#206](https://github.com/rodjek/puppet-lint/pull/206) ([dcarley](https://github.com/dcarley))
|
765
774
|
- incorrect top-scope variable warning for define [\#205](https://github.com/rodjek/puppet-lint/pull/205) ([blalor](https://github.com/blalor))
|
766
775
|
- --fix doesn't modify my file [\#203](https://github.com/rodjek/puppet-lint/pull/203) ([BillWeiss](https://github.com/BillWeiss))
|
767
776
|
- Don't warn about 'mode' format when it's an 'audit' value [\#199](https://github.com/rodjek/puppet-lint/pull/199) ([bitfield](https://github.com/bitfield))
|
@@ -769,8 +778,6 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
769
778
|
- Running with --fix deletes all code for files containing invalid syntax [\#184](https://github.com/rodjek/puppet-lint/pull/184) ([devicenull](https://github.com/devicenull))
|
770
779
|
- Puppet-lint fails to handle mac line endings [\#183](https://github.com/rodjek/puppet-lint/pull/183) ([devicenull](https://github.com/devicenull))
|
771
780
|
- Lone dollar sign should not be removed [\#180](https://github.com/rodjek/puppet-lint/pull/180) ([Seldaek](https://github.com/Seldaek))
|
772
|
-
- Fix illegal replacement of double quotes by single quotes [\#179](https://github.com/rodjek/puppet-lint/pull/179) ([Seldaek](https://github.com/Seldaek))
|
773
|
-
- Write files in binary mode to avoid writing CRLFs on windows [\#178](https://github.com/rodjek/puppet-lint/pull/178) ([Seldaek](https://github.com/Seldaek))
|
774
781
|
- Warn about unquoted node names [\#177](https://github.com/rodjek/puppet-lint/pull/177) ([bitfield](https://github.com/bitfield))
|
775
782
|
- Fix the check\_classes check for certain cases. [\#176](https://github.com/rodjek/puppet-lint/pull/176) ([vStone](https://github.com/vStone))
|
776
783
|
- Invalid warning about mandatory class parameters without defaults [\#167](https://github.com/rodjek/puppet-lint/pull/167) ([svend](https://github.com/svend))
|
@@ -790,7 +797,7 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
790
797
|
**Merged pull requests:**
|
791
798
|
|
792
799
|
- Fix where . is located in docs [\#169](https://github.com/rodjek/puppet-lint/pull/169) ([gmjosack](https://github.com/gmjosack))
|
793
|
-
- Update README.md [\#168](https://github.com/rodjek/puppet-lint/pull/168) ([
|
800
|
+
- Update README.md [\#168](https://github.com/rodjek/puppet-lint/pull/168) ([levpaul](https://github.com/levpaul))
|
794
801
|
|
795
802
|
## [0.3.2](https://github.com/rodjek/puppet-lint/tree/0.3.2) (2012-10-19)
|
796
803
|
|
@@ -828,6 +835,10 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
828
835
|
|
829
836
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/0.2.1...0.3.0)
|
830
837
|
|
838
|
+
**Implemented enhancements:**
|
839
|
+
|
840
|
+
- Provide context for the problems [\#130](https://github.com/rodjek/puppet-lint/issues/130)
|
841
|
+
|
831
842
|
**Closed issues:**
|
832
843
|
|
833
844
|
- False positive unquoted resource title for colons in resource parameters [\#146](https://github.com/rodjek/puppet-lint/issues/146)
|
@@ -837,7 +848,6 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
837
848
|
- Crash on \\ at end of single quoted string [\#144](https://github.com/rodjek/puppet-lint/issues/144)
|
838
849
|
- Strings ending in backslash cause exceptions [\#142](https://github.com/rodjek/puppet-lint/issues/142)
|
839
850
|
- --no-class\_parameter\_defaults-check [\#139](https://github.com/rodjek/puppet-lint/issues/139)
|
840
|
-
- Provide context for the problems [\#130](https://github.com/rodjek/puppet-lint/issues/130)
|
841
851
|
|
842
852
|
**Merged pull requests:**
|
843
853
|
|
@@ -859,6 +869,12 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
859
869
|
|
860
870
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/0.2.0.pre1...0.2.0)
|
861
871
|
|
872
|
+
**Implemented enhancements:**
|
873
|
+
|
874
|
+
- Variables standing by themselves should not be quoted [\#20](https://github.com/rodjek/puppet-lint/issues/20)
|
875
|
+
- write class\_parameter\_defaults check [\#134](https://github.com/rodjek/puppet-lint/issues/134)
|
876
|
+
- duplicate parameter detection [\#122](https://github.com/rodjek/puppet-lint/issues/122)
|
877
|
+
|
862
878
|
**Closed issues:**
|
863
879
|
|
864
880
|
- `require': iconv will be deprecated in the future, use String\#encode instead. [\#133](https://github.com/rodjek/puppet-lint/issues/133)
|
@@ -866,14 +882,11 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
866
882
|
- There's a puppet-lint 0.13.1 gem on rubygems.org but no 0.13.1 tag here [\#99](https://github.com/rodjek/puppet-lint/issues/99)
|
867
883
|
- Introduce a way to tweak warning/error level for checks [\#91](https://github.com/rodjek/puppet-lint/issues/91)
|
868
884
|
- Didn't pick up $ipaddress\_bond0 as a fact [\#61](https://github.com/rodjek/puppet-lint/issues/61)
|
869
|
-
- Variables standing by themselves should not be quoted [\#20](https://github.com/rodjek/puppet-lint/issues/20)
|
870
|
-
- write class\_parameter\_defaults check [\#134](https://github.com/rodjek/puppet-lint/issues/134)
|
871
885
|
- False positive when using function call for parameter default [\#132](https://github.com/rodjek/puppet-lint/issues/132)
|
872
886
|
- Fix up the website [\#131](https://github.com/rodjek/puppet-lint/issues/131)
|
873
887
|
- Linked list style functionality for tokens [\#129](https://github.com/rodjek/puppet-lint/issues/129)
|
874
888
|
- False positive unquoted resource title in case statements [\#128](https://github.com/rodjek/puppet-lint/issues/128)
|
875
889
|
- Crash on arrow alignment check [\#127](https://github.com/rodjek/puppet-lint/issues/127)
|
876
|
-
- duplicate parameter detection [\#122](https://github.com/rodjek/puppet-lint/issues/122)
|
877
890
|
- Does not work on ruby 1.9.3 [\#120](https://github.com/rodjek/puppet-lint/issues/120)
|
878
891
|
- Bad class format causes exception [\#118](https://github.com/rodjek/puppet-lint/issues/118)
|
879
892
|
- case statement in inline\_template false warning [\#117](https://github.com/rodjek/puppet-lint/issues/117)
|
@@ -918,6 +931,7 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
918
931
|
- Fix utf8 char issues: see bug \#84 [\#108](https://github.com/rodjek/puppet-lint/pull/108) ([vStone](https://github.com/vStone))
|
919
932
|
- Add support for ignoring certain globs in the rake task [\#106](https://github.com/rodjek/puppet-lint/pull/106) ([wfarr](https://github.com/wfarr))
|
920
933
|
- Fix bug introduced in \#81 [\#98](https://github.com/rodjek/puppet-lint/pull/98) ([deizel](https://github.com/deizel))
|
934
|
+
- Fixup the variable not enclosed in {} test [\#66](https://github.com/rodjek/puppet-lint/pull/66) ([richardc](https://github.com/richardc))
|
921
935
|
|
922
936
|
## [0.1.13](https://github.com/rodjek/puppet-lint/tree/0.1.13) (2012-03-26)
|
923
937
|
|
@@ -937,7 +951,6 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
937
951
|
- Add support run puppet-lint on directory. [\#81](https://github.com/rodjek/puppet-lint/pull/81) ([nanliu](https://github.com/nanliu))
|
938
952
|
- Update travis support for multiple puppet version. [\#80](https://github.com/rodjek/puppet-lint/pull/80) ([nanliu](https://github.com/nanliu))
|
939
953
|
- Find booleans in double quoted strings [\#67](https://github.com/rodjek/puppet-lint/pull/67) ([richardc](https://github.com/richardc))
|
940
|
-
- Fixup the variable not enclosed in {} test [\#66](https://github.com/rodjek/puppet-lint/pull/66) ([richardc](https://github.com/richardc))
|
941
954
|
- Feature/symbolic filemodes \(cfr ticket \#60\) [\#62](https://github.com/rodjek/puppet-lint/pull/62) ([vStone](https://github.com/vStone))
|
942
955
|
|
943
956
|
## [0.1.12](https://github.com/rodjek/puppet-lint/tree/0.1.12) (2012-01-27)
|
@@ -1046,7 +1059,7 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
1046
1059
|
|
1047
1060
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/0.0.7...0.1.0)
|
1048
1061
|
|
1049
|
-
**
|
1062
|
+
**Implemented enhancements:**
|
1050
1063
|
|
1051
1064
|
- display order of class/define parameters [\#19](https://github.com/rodjek/puppet-lint/issues/19)
|
1052
1065
|
- class inheritance [\#18](https://github.com/rodjek/puppet-lint/issues/18)
|
@@ -1054,6 +1067,9 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
1054
1067
|
- relationship declarations [\#16](https://github.com/rodjek/puppet-lint/issues/16)
|
1055
1068
|
- defaults for case statements and selectors [\#15](https://github.com/rodjek/puppet-lint/issues/15)
|
1056
1069
|
- namespacing variables [\#14](https://github.com/rodjek/puppet-lint/issues/14)
|
1070
|
+
|
1071
|
+
**Closed issues:**
|
1072
|
+
|
1057
1073
|
- =\> alignment warnings in selectors is broken [\#11](https://github.com/rodjek/puppet-lint/issues/11)
|
1058
1074
|
|
1059
1075
|
## [0.0.7](https://github.com/rodjek/puppet-lint/tree/0.0.7) (2011-08-21)
|
@@ -1064,7 +1080,7 @@ If the additional gems you use for checks are pinned to 1.x, you should pin pupp
|
|
1064
1080
|
|
1065
1081
|
[Full Changelog](https://github.com/rodjek/puppet-lint/compare/0.0.5...0.0.6)
|
1066
1082
|
|
1067
|
-
**
|
1083
|
+
**Implemented enhancements:**
|
1068
1084
|
|
1069
1085
|
- Please add logic to the \>80 chars check [\#12](https://github.com/rodjek/puppet-lint/issues/12)
|
1070
1086
|
|