voxpupuli / puppet-lint-strict_indent-check

indent check for puppet-lint
Mozilla Public License 2.0
4 stars 10 forks source link

Heredoc did not detected #25

Closed jkroepke closed 1 year ago

jkroepke commented 2 years ago

Hi,

I'm currently integrate puppet-lint-strict_indent-check but it currently detect one heredoc syntax not correctly.

  concat::fragment { '/etc/amavis/conf.d/50-user/dkim':
    target  => '/etc/amavis/conf.d/50-user',
    order   => '10',
    content => @(FILE),
      $signed_header_fields{'received'} = 0;
      $enable_dkim_verification = 1;
      $enable_dkim_signing = 1;
      @client_ipaddr_policy = (
          \@mynetworks => 'MYNETS',
      );
      $policy_bank{'MYNETS'} = {
          originating => 1,
          final_virus_destiny => D_REJECT,
          final_spam_destiny => D_REJECT,
          final_bad_header_destiny => D_PASS,
          terminate_dsn_on_notify_success => 0,
          warnbadhsender => 1,
      };
      | FILE
    ;
  }

The error I get:

postfix/amavis.pp:160:strict_indent:WARNING:indent should be 6 chars and is 0

  $signed_header_fields{'received'} = 0;
  ^

Other heredoc location are fine. I guess something detected wrong, if there is a $. I add a case here as test.

codecov[bot] commented 2 years ago

Codecov Report

Merging #25 (871476e) into main (654fb10) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   92.10%   92.10%           
=======================================
  Files           1        1           
  Lines          76       76           
=======================================
  Hits           70       70           
  Misses          6        6           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 654fb10...871476e. Read the comment docs.