Skip to content

Add alignment for hashes and selectors#259

Open
gerases wants to merge 1 commit intopuppetlabs:mainfrom
gerases:improve-arrow-alignment-plugin
Open

Add alignment for hashes and selectors#259
gerases wants to merge 1 commit intopuppetlabs:mainfrom
gerases:improve-arrow-alignment-plugin

Conversation

@gerases
Copy link

@gerases gerases commented Jan 14, 2026

Summary

The change adds arrow alignment for hashes and selectors, which from what I can tell has been missing. For example this code would pass the check with the current code:

$x = {
  'a' => 1,
  'b'  => 2,
}

or this code:

$rootgroup = $facts['os']['family'] ? {
  'RedHat'                     => 'wheel',
  /(Debian|Ubuntu)/  => 'wheel',
  default                      => 'root',
}

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • Thought process behind the implementation.

Related Issues (if any)

#226

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.\

@gerases gerases requested review from a team and bastelfreak as code owners January 14, 2026 19:12
@bastelfreak
Copy link
Collaborator

Hi, did you check if https://github.com/voxpupuli/puppet-lint-strict_indent-check does already what you want?

@gerases gerases force-pushed the improve-arrow-alignment-plugin branch 4 times, most recently from 0120944 to 11ada56 Compare January 14, 2026 19:59
@gerases gerases force-pushed the improve-arrow-alignment-plugin branch from 11ada56 to 9c17259 Compare January 14, 2026 20:04
@gerases
Copy link
Author

gerases commented Jan 14, 2026

Hi, did you check if https://github.com/voxpupuli/puppet-lint-strict_indent-check does already what you want?

Yes, it doesn't do this unfortunately

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