Skip to content

puppet-lint fixes - #632

Closed
pearsondavid wants to merge 1 commit into
puppetlabs:mainfrom
pearsondavid:puppetlintfixes
Closed

puppet-lint fixes#632
pearsondavid wants to merge 1 commit into
puppetlabs:mainfrom
pearsondavid:puppetlintfixes

Conversation

@pearsondavid

Copy link
Copy Markdown

tidy manifests to comply with puppet-lint

@pearsondavid
pearsondavid requested review from a team and bastelfreak as code owners September 26, 2025 15:11
@CLAassistant

CLAassistant commented Sep 26, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

tidy manifests to comply with puppet-lint
@LukasAud

LukasAud commented Aug 3, 2026

Copy link
Copy Markdown

Hi @pearsondavid, I had to start a rebase to kick off testing. Not sure if the PR has been affected by recent updates or puppet-lint changes but your suggested changes do not pass our puppet-lint automated tests. It looks like the PR could have lost its purpose as time passed.

For now, I'll close it. Feel free to re-open if you think this still fixes a valid problem.

@LukasAud LukasAud closed this Aug 3, 2026
@pearsondavid

pearsondavid commented Aug 3, 2026

Copy link
Copy Markdown
Author

Thanks @LukasAud

Looking a the latest code from v9.1.0, the issue hasn't been fixed in instance_service.pp and program.pp has similar issues now as well. Shall I re-open this and/or create a new one?

./types/programs.pp - ERROR: there should be a single newline at the end of a manifest on line 9 (check: manifest_whitespace_double_newline_end_of_file)
./manifests/program.pp - WARNING: indent should be 6 chars and is 8 on line 44 (check: strict_indent)
./manifests/program.pp - WARNING: indent should be 6 chars and is 8 on line 45 (check: strict_indent)
./manifests/program.pp - WARNING: indent should be 6 chars and is 8 on line 46 (check: strict_indent)
./manifests/program.pp - WARNING: indent should be 6 chars and is 8 on line 47 (check: strict_indent)
./manifests/program.pp - WARNING: indent should be 6 chars and is 8 on line 48 (check: strict_indent)
./manifests/instance_service.pp - WARNING: indent should be 4 chars and is 6 on line 50 (check: strict_indent)
./manifests/instance_service.pp - WARNING: indent should be 4 chars and is 6 on line 55 (check: strict_indent)
./manifests/instance_service.pp - WARNING: indent should be 4 chars and is 6 on line 56 (check: strict_indent)
./manifests/instance_service.pp - WARNING: indent should be 4 chars and is 6 on line 57 (check: strict_indent)
./manifests/instance_service.pp - WARNING: indent should be 4 chars and is 6 on line 58 (check: strict_indent)

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.

4 participants