Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Remove the deprecated spellings, and --output-file, which no tool uses

Aperta
#173 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
48/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
cpp

Direzione di ricerca

Start with the deprecated option and key handling in src/App/Config.cpp, src/App/ToolConfig.cpp, and include/ctrace_defs/types.hpp; review the tests and docs/configuration.md section named in the issue. Update the warning for --output-file and output_file, then remove the listed compatibility paths, their tests, and the Legacy Compatibility section. Update CHANGELOG.md for the removal release and run ctest; removed keys and options should be rejected.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

chore severity/minor

ctrace still accepts spellings that were replaced, each with its own code path, tests and documentation:

Spelling Replaced by Where
--async, runtime.async --jobs, runtime.jobs (#158) src/App/Config.cpp:151, src/App/ToolConfig.cpp:729
--ipc=socket --ipc=standardIO (#83) include/ctrace_defs/types.hpp:26-41
Root invoke and input, stack_analyzer legacy keys, stack_analyzer.include_dirs and defines, tools.ctrace_stack_analyzer, tools.stack_analyzer The canonical sections (#87) src/App/ToolConfig.cpp (applyLegacyRootInvokeAndInput, deprecated); docs/configuration.md, "Legacy Compatibility"

--output-file (config output.output_file, API output_file) is in a worse state: its help text says "Legacy output path; no current tool writes to it" (src/App/Config.cpp:72). It is accepted, without any warning, and does nothing.

To do

  1. Make --output-file and output_file warn like the other deprecated spellings.
  2. In CHANGELOG.md, announce the release that removes them all.
  3. In that release, remove them, with their tests and the "Legacy Compatibility" section of docs/configuration.md.

Acceptance

  • After the removal, a removed key is rejected with the unknown-key diagnostic that names the allowed keys, and a removed option is rejected by the CLI.
  • ctest passes.

From the technical debt register (docs/quality/technical-debt.md, D6).

Lingua principale
C++
Stelle
5
Fork
4
Merge medio
1h 57m
PR unite (30g)
88

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di CoreTrace/coretrace

Tutte le issue di CoreTrace/coretrace

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.