puppet-lint 2.4.2 → 2.5.0
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 +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
|
|