Skip to content

Replace Codecov with Qlty for code quality and coverage#387

Open
akostadinov-bot[bot] wants to merge 4 commits into
mainfrom
replace-codecov-with-qlty
Open

Replace Codecov with Qlty for code quality and coverage#387
akostadinov-bot[bot] wants to merge 4 commits into
mainfrom
replace-codecov-with-qlty

Conversation

@akostadinov-bot

Copy link
Copy Markdown

Summary

  • Bootstrapped qlty config via qlty init, added reek plugin, pinned rubocop to 1.75.2 (matching pisoni setup)
  • Replaced deprecated codecov gem with simplecov + simplecov_json_formatter
  • Updated spec/spec_helper.rb to use COVERAGE env var with JSON+HTML multi-formatter (matching pisoni pattern)
  • Added COVERAGE=1 to CI unit test step and qlty coverage upload step after tests

Test plan

  • bundle install succeeds
  • COVERAGE=1 bundle exec rake spec:unit runs and generates coverage/.resultset.json
  • CI build passes and uploads coverage to Qlty
  • Qlty dashboard shows coverage data for this project

🤖 Generated with Claude Code

Bob Dev and others added 2 commits July 3, 2026 21:39
- Add qlty config via `qlty init` with reek and pinned rubocop
- Replace codecov gem with simplecov + simplecov_json_formatter
- Update spec_helper to use COVERAGE env var with JSON+HTML formatters
- Add coverage upload step to CircleCI after unit tests

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Remove deprecated workflows version: 2 key. PR branches require
manual approval before all jobs run. Main branch runs automatically.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@qltysh

qltysh Bot commented Jul 3, 2026

Copy link
Copy Markdown

❌ 3 blocking issues (3 total)

Tool Category Rule Count
rubocop Style Use 2 spaces for indentation in an array, relative to the first position after the preceding left parenthesis. 2
rubocop Style Incorrect formatting, autoformat by running qlty fmt. 1

@qltysh one-click actions:

  • Auto-fix formatting (qlty fmt && git push)

Comment thread spec/spec_helper.rb
require 'simplecov_json_formatter'
SimpleCov.start do
formatter SimpleCov::Formatter::MultiFormatter.new([
SimpleCov::Formatter::JSONFormatter,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Use 2 spaces for indentation in an array, relative to the first position after the preceding left parenthesis. [rubocop:Layout/FirstArrayElementIndentation]

Comment thread spec/spec_helper.rb
formatter SimpleCov::Formatter::MultiFormatter.new([
SimpleCov::Formatter::JSONFormatter,
SimpleCov::Formatter::HTMLFormatter
])

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Indent the right bracket the same as the first position after the preceding left parenthesis. [rubocop:Layout/FirstArrayElementIndentation]

Bob Dev and others added 2 commits July 3, 2026 21:49
- Add use-https-checkout command to avoid SSH auth failures
- Upload coverage with --tag unit and --tag integration separately
- Mark uploads as --incomplete, finalize with complete-coverage job
- Add integration_tests to main workflow for coverage on main branch

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Deploy key will be used instead.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

0 participants