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

Fix Rubocop convention offenses #4735

Merged
merged 2 commits into from
Nov 16, 2021
Merged

Fix Rubocop convention offenses #4735

merged 2 commits into from
Nov 16, 2021

Conversation

javierm
Copy link
Member

@javierm javierm commented Nov 10, 2021

References

Objectives

  • Have zero reports about offenses when running rubocop --fail-level convention --display-only-fail-level-offenses
  • Keep the code as clean and consistent as possible

It was accidentally introduced in commit 2b709f1. Pronto didn't warn
us because the blank lines were together after removing the blank lines
between them.
It was accidentally introduced in commit 756a16f. Pronto didn't warn
us because in that commit we deleted the code where the `group` method
was used.
@javierm javierm added Linters Rubocop, ERB Lint, ESLint, SCSS-Lint, ... 1.4 labels Nov 10, 2021
@javierm javierm self-assigned this Nov 10, 2021
@javierm javierm changed the title Fix Rubocop offenses Fix Rubocop convention offenses Nov 10, 2021
@taitus taitus self-assigned this Nov 16, 2021
@javierm javierm removed the 1.4 label Nov 16, 2021
@javierm javierm merged commit b548e16 into master Nov 16, 2021
@javierm javierm deleted the rubocop_offenses_1.4 branch November 16, 2021 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Linters Rubocop, ERB Lint, ESLint, SCSS-Lint, ...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants