False alignment positives

[adamgibbins]

Getting loads of them since (I think) a recent update.
Examples:

 177   File {
 178     owner   => root,
 179     group   => root,
 180   }

WARNING: => on line 179 isn't aligned with the previous line

 135     ensure => $ensure ? {
 136       present => directory,
 137       absent  => undef,
 138     },
 139     owner  => 'foo4',
 140     group  => 'foo4',
 141     mode   => '0755',

WARNING: => on line 140 isn't aligned with the previous line
WARNING: => on line 141 isn't aligned with the previous line

  80   file { '/foo/.bar':
  81     source  => 'puppet:///modules/foo/.bar',
  82     owner   => 'root',
  83     group   => 'foo4',
  84     require => Class['foo::install']
  85   }

WARNING: => on line 82 isn't aligned with the previous line
WARNING: => on line 83 isn't aligned with the previous line
WARNING: => on line 84 isn't aligned with the previous line

Tested using Lint 0.1.0, Puppet 2.7.3

[hkraal]

The first to examples should be correct since it isn't correctly alligned as I understand. Within the definition you take the longest parameter, place 1 space after it and align the rest to that one. The first to should be like:

 177   File {
 178     owner => root,
 179     group => root,
 180   }

and the second:

  80   file { '/foo/.bar:
  81     source  => 'puppet:///modules/foo/.bar',
  82     owner   => 'root',
  83     group   => 'foo4',
  84     require => Class['foo::install']
  85   }

This, however doesn't make sense to the 3rd example but since it contain's a syntax error I suppose that the example is invalid.

[adamgibbins]

Syntax error is a typo, it still occurs even when corrected.

RE the first issue, you could be correct (if you aren't you should be), my mistake, bad example.

[hkraal]

Hmmm not sure but I aint receiving an error now when I copy/paste?

user@host:~# cat test.pp
file { '/foo/.bar':
  source  => 'puppet:///modules/foo/.bar',
  owner   => 'root',
  group   => 'foo4',
  require => Class['foo::install']
}

user@host:~# puppet-lint test.pp
user@host:~# md5sum test.pp
d9e51c4d5f2f945d5d3631d27016aed5  test.pp
user@host:~#

[rodjek]

So, this should be fixed in 6d26876 (v0.1.3). Give it a shot and let me know.