Skip to content

Fix all RuboCop offenses and add CI workflow#1

Merged
moviendome merged 1 commit intomainfrom
rubocop-ci-setup
Mar 21, 2026
Merged

Fix all RuboCop offenses and add CI workflow#1
moviendome merged 1 commit intomainfrom
rubocop-ci-setup

Conversation

@moviendome
Copy link
Copy Markdown
Member

Summary

  • Resolve all 289 RuboCop offenses (0 remaining)
  • Add .rubocop.yml with NewCops: enable
  • Refactor controller, verifier, and generator to reduce method complexity
  • Split large test classes for maintainability
  • Add GitHub Actions CI: tests on Ruby 3.2/3.3/3.4 matrix + RuboCop lint job
  • Add rubocop to Gemfile

Test plan

  • All 37 tests pass (79 assertions, 0 failures)
  • RuboCop reports 0 offenses across 23 files
  • CI runs green on GitHub Actions

🤖 Generated with Claude Code

- Use rubocop-rails-omakase for Rails-standard style rules
- Add GitHub Actions CI: tests on Ruby 3.2/3.3/3.4 + RuboCop job
- Fix 6 minor formatting offenses (array bracket spacing, empty line)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@moviendome moviendome merged commit 8b6511b into main Mar 21, 2026
4 checks passed
@moviendome moviendome deleted the rubocop-ci-setup branch March 21, 2026 11:07
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.

2 participants