puppet-lint 0.1.3 → 0.1.4

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.
data/README.md CHANGED
@@ -38,6 +38,7 @@ At the moment, the following tests have been implemented:
38
38
 
39
39
  * All strings that do not contain variables should be enclosed in single
40
40
  quotes.
41
+ * An exception has been made for double quoted strings containing \n or \t.
41
42
  * All strings that contain variables must be enclosed in double quotes.
42
43
  * All variables should be enclosed in braces when interpolated in a string.
43
44
  * Variables standing by themselves should not be quoted.
data/lib/puppet-lint.rb CHANGED
@@ -5,7 +5,7 @@ require 'puppet'
5
5
  class PuppetLint::NoCodeError < StandardError; end
6
6
 
7
7
  class PuppetLint
8
- VERSION = '0.1.3'
8
+ VERSION = '0.1.4'
9
9
 
10
10
  attr_reader :code, :file
11
11
 
@@ -25,7 +25,9 @@ class PuppetLint::Plugins::CheckStrings < PuppetLint::CheckPlugin
25
25
  token = tokens[token_idx]
26
26
 
27
27
  if token.first == :STRING
28
- warn "double quoted string containing no variables on line #{token.last[:line]}"
28
+ unless token.last[:value].include? "\t" or token.last[:value].include? "\n"
29
+ warn "double quoted string containing no variables on line #{token.last[:line]}"
30
+ end
29
31
  end
30
32
 
31
33
  if token.first == :DQPRE and token.last[:value] == ""
data/puppet-lint.gemspec CHANGED
@@ -1,6 +1,6 @@
1
1
  Gem::Specification.new do |s|
2
2
  s.name = 'puppet-lint'
3
- s.version = '0.1.3'
3
+ s.version = '0.1.4'
4
4
  s.homepage = 'https://github.com/rodjek/puppet-lint/'
5
5
  s.summary = 'Ensure your Puppet manifests conform with the Puppetlabs style guide'
6
6
  s.description = 'Checks your Puppet manifests against the Puppetlabs
@@ -48,4 +48,11 @@ describe PuppetLint::Plugins::CheckStrings do
48
48
  its(:warnings) { should be_empty }
49
49
  its(:errors) { should be_empty }
50
50
  end
51
+
52
+ describe 'double quoted string containing newline but no variables' do
53
+ let(:code) { '"foo\n"' }
54
+
55
+ its(:warnings) { should be_empty }
56
+ its(:errors) { should be_empty }
57
+ end
51
58
  end
metadata CHANGED
@@ -1,13 +1,13 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: puppet-lint
3
3
  version: !ruby/object:Gem::Version
4
- hash: 29
4
+ hash: 19
5
5
  prerelease:
6
6
  segments:
7
7
  - 0
8
8
  - 1
9
- - 3
10
- version: 0.1.3
9
+ - 4
10
+ version: 0.1.4
11
11
  platform: ruby
12
12
  authors:
13
13
  - Tim Sharpe