Skip to content

fix(lint): autocorrect simple style and layout violations#388

Merged
natsuki615 merged 2 commits intomasterfrom
fix/lint-simple-autocorrect
Mar 6, 2026
Merged

fix(lint): autocorrect simple style and layout violations#388
natsuki615 merged 2 commits intomasterfrom
fix/lint-simple-autocorrect

Conversation

@merichar
Copy link
Copy Markdown
Member

@merichar merichar commented Mar 2, 2026

  • collapse empty method definitions onto a single line (Style/EmptyMethod, 18)
  • remove extra blank lines (Layout/EmptyLines, 13)
  • convert String#% formatting to format() calls (Style/FormatString, 2)
  • convert remaining non-interpolated double-quoted strings to single quotes (Style/StringLiterals, 2)
  • move top-level include Messenger into Event, OrganizationTimelineEntry, and Shift class bodies
    (Style/MixinUsage)
  • move top-level include WebMock::API into ActiveSupport::TestCase in test_helper (Style/MixinUsage)
  • wrap long commented-out lines in db/seeds.rb (Layout/LineLength)

merichar added 2 commits March 1, 2026 20:38
* collapse empty method definitions to single line (Style/EmptyMethod)
* remove extra blank lines (Layout/EmptyLines)
* convert String#% to format() (Style/FormatString)
* convert remaining double-quoted strings to single quotes (Style/StringLiterals)
… line-length

* move include Messenger inside Event, OrganizationTimelineEntry, and Shift classes
* move include WebMock::API inside ActiveSupport::TestCase in test_helper
* wrap long commented-out lines in db/seeds.rb
@natsuki615 natsuki615 merged commit b88e28c into master Mar 6, 2026
1 check failed
@merichar merichar deleted the fix/lint-simple-autocorrect branch March 19, 2026 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants