Skip to content

Conversation

@ekohl
Copy link
Member

@ekohl ekohl commented Oct 19, 2022

The goal here was to show somethign that I think is currently failing but should pass. However, puppet-lint 3.0.0 also broke our tests (puppetlabs/puppet-lint#65).

@codecov
Copy link

codecov bot commented Oct 19, 2022

Codecov Report

Base: 92.10% // Head: 0.00% // Decreases project coverage by -92.10% ⚠️

Coverage data is based on head (a8cc8e4) compared to base (654fb10).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #28       +/-   ##
==========================================
- Coverage   92.10%   0.00%   -92.11%     
==========================================
  Files           1       1               
  Lines          76     135       +59     
==========================================
- Hits           70       0       -70     
- Misses          6     135      +129     
Impacted Files Coverage Δ
lib/puppet-lint/plugins/check_strict_indent.rb 0.00% <0.00%> (-92.11%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@bastelfreak bastelfreak marked this pull request as ready for review October 2, 2025 19:15
$groups = if $arg1 { ['wheel'] } else { [] }

$nested_groups = if $arg1 {
['wheel']
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ekohl I think it's correct that this fails, the indent is 6, but it should be 4? same on the following three lines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants