Repository navigation
style: aplica prettier em alerts.store.ts e customers.store.ts - #3
Merged
Merged
Conversation
O commit e6eb99e do upstream deixou esses dois arquivos fora do padrão do prettier, e o format-check do tests.yml falha em qualquer PR para o master. Mudança só de formatação (npx prettier --write), sem alteração de lógica. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SoGzgXQF8Sf1Jsf6n7ZNia
O deploy.yml publica na infraestrutura do alerta.io (Fly.io no servidor; GitHub Pages com endpoint alerta-api.fly.dev e tracking UA-44644195-1 na interface). No fork isso falharia ou publicaria algo errado a cada push no master, e na interface o job de deploy roda também em pull_request. Passa a ser só workflow_dispatch, até termos destino e segredos próprios. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SoGzgXQF8Sf1Jsf6n7ZNia (cherry picked from commit 7c77b26)
rodolphobrock
marked this pull request as ready for review
September 29, 2026 13:08
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O
format-checkdotests.yml(prettier --check '**/*.ts') falha nomasterdesde o commite6eb99edo upstream, que deixou dois arquivos fora do padrão:src/store/modules/alerts.store.tssrc/store/modules/customers.store.tsCom isso, o job
lintfica vermelho em qualquer PR para omaster, com ou sem relação com esses arquivos.Mudança
npx prettier --writenos dois arquivos (prettier 2.8.8, a versão dopackage.json).{ ALL: 0 }→{ALL: 0}). Nenhuma mudança de lógica.Validação
npm run format-check: "All matched files use Prettier code style!"npm run lint:nofix: 0 erros; os avisos já existiam antes.Onde mais a correção entrou
O mesmo commit foi aplicado com
cherry-pick -xnos branches dos PRs que estavam vermelhos por causa disso:Quando este PR entrar no
master, o commit repetido nesses branches não muda mais nada.Vale também mandar esta correção para o upstream, porque o
masterdeles está vermelho pelo mesmo motivo.🤖 Generated with Claude Code
https://claude.ai/code/session_01SoGzgXQF8Sf1Jsf6n7ZNia
Generated by Claude Code