Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(maint) Fix rubocop lint warnings #1298

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Conversation

gavindidrichsen
Copy link
Contributor

Summary

Nightly CI is failing with the following rubocop warnings:

Run bundle exec rubocop --format github
  bundle exec rubocop --format github
  shell: /usr/bin/bash -e {0}
  env:
    PUPPET_GEM_VERSION: ~> 7.0

Error: Lint/SymbolConversion: Unnecessary symbol conversion; use `:"write_#{format}"` instead.
Error: Style/HashEachMethods: Use `each_value` instead of `each` and remove the unused `_` block argument.
Error: Style/HashEachMethods: Use `each_value` instead of `each` and remove the unused `_tool` block argument.
Error: Style/RedundantParentheses: Don't use parentheses around a logical expression.
Error: Style/HashEachMethods: Use `each_key` instead of `each` and remove the unused `_` block argument.
Error: Style/HashEachMethods: Use `each_key` instead of `each` and remove the unused `_` block argument.
Error: Style/HashEachMethods: Use `each_value` instead of `each` and remove the unused `_source` block argument.
Error: Process completed with exit code 1.

Linting has been fixed and comments added where appropriate.

Checklist

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

@gavindidrichsen gavindidrichsen added the maintenance Internal maintenance work that shouldn't appear in the changelog label Dec 4, 2023
@gavindidrichsen gavindidrichsen marked this pull request as ready for review December 4, 2023 18:18
@gavindidrichsen gavindidrichsen requested a review from a team as a code owner December 4, 2023 18:18
Signed-off-by: Gavin Didrichsen <[email protected]>
Copy link
Contributor

@jordanbreen28 jordanbreen28 left a comment

Choose a reason for hiding this comment

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

LGTM

@jordanbreen28 jordanbreen28 merged commit 57c39ca into main Dec 5, 2023
8 checks passed
@jordanbreen28 jordanbreen28 deleted the maint_fix_rubocop_warnings branch December 5, 2023 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Internal maintenance work that shouldn't appear in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants