Skip to content

Remove qlty integration from CI - #284

Merged
gabrieldeespindula merged 1 commit into
masterfrom
remove-qlty-ci
Aug 20, 2026
Merged

Remove qlty integration from CI#284
gabrieldeespindula merged 1 commit into
masterfrom
remove-qlty-ci

Conversation

@pnzrfst

@pnzrfst pnzrfst commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Resumo

  • Remove os steps de instalação e publicação do qlty (curl https://qlty.sh | sh + qlty coverage publish) do job rspec em .github/workflows/config.yml.
  • Reverte spec/spec_helper.rb para gerar cobertura só em HTML (o formatter JSON só existia para alimentar o qlty coverage publish).
  • Remove a gem simplecov-json, agora sem uso.

Por quê

O projeto br_danfe no Qlty Cloud está pausado, e qlty coverage publish trata "paused project" como erro fatal (HTTP 400), quebrando o CI mesmo com a suíte passando 100%. Confirmado ao rerodar as PRs #280 e #281: ambas com testes verdes, falhando só nesse step.

O facil123 já tinha cortado o qlty pelo mesmo motivo (redundante com o couve, que já fazia o gate de cobertura + comentário na PR).

Fecha #283.

Plano de teste

  • bundle exec rspec local: 291 examples, 0 failures
  • bundle exec rubocop --display-cop-names --parallel local: sem ofensas
  • CI verde nesta PR (sem o step do qlty)

🤖 Generated with Claude Code

The br_danfe project on Qlty Cloud is paused, so `qlty coverage
publish` hard-fails CI (HTTP 400) even when the test suite passes.
facil123 already dropped qlty in favor of couve for the same reason
(no one was consuming the qlty.sh dashboard).

Reverts the CI/coverage-formatter changes from Setup Qlty (#254) and
its follow-up (#255).

Refs #283

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@pnzrfst pnzrfst self-assigned this Aug 18, 2026
@pnzrfst pnzrfst linked an issue Aug 18, 2026 that may be closed by this pull request
@gabrieldeespindula
gabrieldeespindula merged commit 7e75d47 into master Aug 20, 2026
2 checks passed
@gabrieldeespindula
gabrieldeespindula deleted the remove-qlty-ci branch August 20, 2026 12:10
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.

Remover integração com qlty do CI (projeto pausado no Qlty Cloud)

2 participants