[BUG][code-analyzer] Disabled rules are moved to the end of the config file rules list
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 58/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- typescript
- Área
- cli
Línea de trabajo
Comienza con el comando sf code-analyzer config y reproduce el problema usando el comando proporcionado y code-analyzer.yml. Inspecciona la lista de reglas pmd generada y verifica después que todas las reglas, incluidas las que tienen disabled: true, estén ordenadas alfabéticamente por clave única y sigan estándolo después de volver a generarlas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Have you tried to resolve this issue yourself first?
- I confirm I have gone through the above steps and still have an issue to report.
Bug Description
I disabled some rules and recreated the config file. ( see #2013 )
All the disabled rules were moved to the end of the pmd section.
But I want to view ALL the PMD rules alphabetized so I know which ones are in there.
Output / Logs
Steps To Reproduce
- Disabled a rule with
disabled: true - Run
echo 'y' | sf code-analyzer config --include-unmodified-rules --rule-selector all --config-file code-analyzer.yml --output-file code-analyzer.yml - See that the disabled rules are moved.
Expected Behavior
All rules should be alphabetized by their unique key. This extra sort doesn't help anything and only can confuse people.
Operating System
macOS
Salesforce CLI Version
@salesforce/cli/2.128.5 darwin-arm64 node-v22.22.1
Code Analyzer Plugin (code-analyzer) Version
5.11.1
Node Version
No response
Java Version
No response
Python Version
No response
Additional Context (Screenshots, Files, etc)
This is actually a significant bug because changing from expected behavior will confuse people.
Workaround
No response
Urgency
Moderate
- Lenguaje dominante
- TypeScript
- Estrellas
- 240
- Forks
- 52
- Merge medio
- 9 h 48 min
- PR fusionados (30 d)
- 3
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 forcedotcom/code-analyzer
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
forcedotcom/code-analyzer#2094 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
forcedotcom/code-analyzer#2093 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
forcedotcom/code-analyzer#2091 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
forcedotcom/code-analyzer#2090 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
forcedotcom/code-analyzer#2099 ·
Todos los issues de forcedotcom/code-analyzer
Issues similares
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Crush Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
ElementsProject/cln-application#167 · 1 comentario · 1 reacción ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Quantco/pnpm-licenses#17 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100