release runner: refuse a changelog entry that a second Prettier pass changes

Aberta Para iniciantes
  • Algo errado?
#588 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
78/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
javascript, node.js
Domínio
tooling

Direção de pesquisa

Comece em scripts/release-please-runner.js e leia formatChangelogMarkdown; em seguida, consulte as orientações relacionadas em scripts/README.md e DEVELOPMENT.md. Reproduza o problema com o trecho fornecido depois de npm --prefix scripts ci; adicione cobertura para entradas que nunca terminam, com e sem *. O trabalho estará concluído quando o runner recusar a entrada e indicar a primeira linha alterada, e as orientações não afirmarem mais quais textos terminam.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

needs-triage

The release runner can write a changelog entry that a second Prettier pass changes.

Defect. scripts/release-please-runner.js formats the new CHANGELOG entry with one Prettier pass. For some entries a second pass changes the text again. mise run format-check then fails on the release PR, and mise run format changes the words of the entry.

Reproduction (main, after npm --prefix scripts ci):

const r = require("./scripts/release-please-runner.js");
const a = r.formatChangelogMarkdown("## [0.4.0]\n\n- Fixed open_in_new_tab when *focus* is false\n");
// a, line 3:           "- Fixed open_in_new_tab when _focus_ is false"
// second pass, line 3: "- Fixed open*in_new_tab when \\_focus* is false"
r.formatChangelogMarkdown(a) === a; // false

#567 only documents this: its refusal message, scripts/README.md and DEVELOPMENT.md tell a maintainer to review the diff, fix such a line by hand, and run mise run format until it changes nothing.

Planned fix. Run the formatter a second time on the new entry and refuse with an error that names the first line the second pass changes (one more Prettier call per update). Trim the README and refusal text so they claim nothing about which texts settle. Test with a never-settling entry with and without *.

Owner: the next change to the release runner. Deferred from the README verification run with the maintainer's agreement.

Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high

Linguagem predominante
Lua
Estrelas
3.2k
Forks
223
Merge médio
6h 7min
PRs com merge (30d)
210

Preparar o ambiente

Abrir no Codespaces

Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.

  • Sem Dockerfile nem arquivo Docker Compose
  • Sem modelo de pull request
  • Sem guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de coder/claudecode.nvim

Todas as issues de coder/claudecode.nvim

Issues semelhantes

Mais issues de Lua

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.