Skip to content

Fix arrow_alignment violation in mysqlbackup.pp - #1732

Merged
LukasAud merged 1 commit into
mainfrom
fix/mysqlbackup-arrow-alignment
Aug 18, 2026
Merged

Fix arrow_alignment violation in mysqlbackup.pp#1732
LukasAud merged 1 commit into
mainfrom
fix/mysqlbackup-arrow-alignment

Conversation

@bronachfalls

Copy link
Copy Markdown
Contributor

The require parameter in the mysql_user resource block was missing alignment spaces, causing puppet-lint's arrow_alignment check to fail:

indentation of => is not properly aligned (expected in column 19,
but found it in column 13) on line 44 (check: arrow_alignment)

All => arrows within a resource block must align at the same column. The other two parameters in this block already have => at column 19:

ensure        => $ensure,
password_hash => Deferred('mysql::password', [$backuppassword]),
require => Class['mysql::server::root_password'],  ← col 13 (wrong)

Restored the correct spacing so all three arrows align at column 19:

require       => Class['mysql::server::root_password'],

Summary

Provide a detailed description of all the changes present in this pull request.

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)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

The `require` parameter in the `mysql_user` resource block was missing
alignment spaces, causing puppet-lint's `arrow_alignment` check to fail:

    indentation of => is not properly aligned (expected in column 19,
    but found it in column 13) on line 44 (check: arrow_alignment)

All `=>` arrows within a resource block must align at the same column.
The other two parameters in this block already have `=>` at column 19:

    ensure        => $ensure,
    password_hash => Deferred('mysql::password', [$backuppassword]),
    require => Class['mysql::server::root_password'],  ← col 13 (wrong)

Restored the correct spacing so all three arrows align at column 19:

    require       => Class['mysql::server::root_password'],

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@LukasAud LukasAud left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@LukasAud
LukasAud merged commit 76cf3b2 into main Aug 18, 2026
19 of 20 checks passed
@LukasAud
LukasAud deleted the fix/mysqlbackup-arrow-alignment branch August 18, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants