Skip to content

chore: add rubocop and brakeman to CI - #8

Merged
vinimachadosantana merged 1 commit into
developfrom
chore/code-quality
Aug 7, 2026
Merged

chore: add rubocop and brakeman to CI#8
vinimachadosantana merged 1 commit into
developfrom
chore/code-quality

Conversation

@vinimachadosantana

Copy link
Copy Markdown
Owner

What

Wires RuboCop and Brakeman into CI as separate parallel jobs.

Why

Style and security checks belong in the pipeline before feature code exists, so every later PR is held to the same bar rather than having it retrofitted.

How

  • Both gems already ship with Rails 8, this only adds the CI jobs.
  • Separate jobs rather than extra steps, so they run in parallel with the specs and a lint failure doesn't hide a failing test.
  • Brakeman as a Rails-specific static analyser.

Testing

  • bundle exec rubocop clean
  • bundle exec brakeman clean
  • CI green with all three jobs

@vinimachadosantana vinimachadosantana self-assigned this Aug 7, 2026
@vinimachadosantana
vinimachadosantana merged commit cbb0105 into develop Aug 7, 2026
3 checks passed
@vinimachadosantana
vinimachadosantana deleted the chore/code-quality branch August 7, 2026 04:47
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.

1 participant