Remove the deprecated spellings, and --output-file, which no tool uses
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- cpp
- Área
- cli, documentation, testing
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Make
--output-fileandoutput_filewarn like the other deprecated spellings. - In
CHANGELOG.md, announce the release that removes them all. - 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.
ctestpasses.
From the technical debt register (docs/quality/technical-debt.md, D6).
- Lenguaje dominante
- C++
- Estrellas
- 5
- Forks
- 4
- Merge medio
- 1 h 2 min
- PR fusionados (30 d)
- 74
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de CoreTrace/coretrace
-
documentation severity/minor
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
chore good first issue severity/minor
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
build severity/minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
build good first issue severity/minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
test: add unit tests for ctrace::RunDirectoryPosiblemente ocupada @tecnolgd la tomó hace 1 día. Abiertogood first issue test
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
CoreTrace/coretrace#164 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de CoreTrace/coretrace
Issues similares
-
agent:WSL bug linux LOW
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
Build failure: limoAbierto0.kind: build failure
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 Menos de una hora Aptitud para principiantes 72/100
finalburnneo/FBNeo#2803 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
ai_p1 comp-message-queues
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ClickHouse/ClickHouse#125890 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
carla-simulator/carla#9973 ·
Los mantenedores suelen responder en 1 día