[Feature Request] Filename-based suppression per engine or file
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 45/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- typescript, yaml
Piste de recherche
Le point d’entrée est la configuration de la CLI de code-analyzer ; commencez par suivre la manière dont le paramètre YAML de niveau supérieur ignores.files est analysé et appliqué. Comparez les formes demandées au niveau du moteur et par règle, puis définissez une couverture montrant que la suppression des noms de fichiers peut être limitée sans affecter les autres moteurs ou règles.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Product Area
The "code-analyzer" CLI
Your Need or Problem
I like the
ignores:
files:
- "**/utils.js"
feature.
Currently, it exists at the top level, meaning we can only exclude filenames for all engines. I would like to exclude at the engine or file level.
The background: Salesforce provides some stock/standard files (JavaScript, Visualforce, Aura, LWC) that have rule violations in them. Instead of having to fix them all when importing starter metadata, I want to exclude them... but I would like to do that from the config file.
IDEALLY, I could do a suppression PER RULE, but I fear that is too much to ask.
This is resulting in a LOT of false-positives and noise, and is stopping our pipeline every time we create a new kind of Experience Cloud site.
Your Desired Solution
engines:
pmd:
ignores:
- "selfRegister.cmp"
- "CommunityTemplate.page"
or
rules:
pmd:
"LoadCSSApexStylesheet":
severity: 2
ignores:
- "CommunityTemplate.page"
tags:
- AppExchange
- Security
- Visualforce
My vision for using Code Analyzer is that I have a standard config file I use for all my projects as a starting point, and that one would include suppressing checks on those files. I could load that as a default in each of my repos/projects.
Alternatives Considered
PMD doesn't have a robust "ignore this rule for this filepath" from the ruleset feature, so I know I'm asking for a lot here.
I can exclude files in my custom ruleset, but that only applies to pmd rules.
I can override the specific rule and make a very long XPath exception property, which works, and might be good for some files, but isn't ideal.
Additional Context (Screenshots, Files, etc)
No response
Workaround
I have been able to suppress for most files (not JavaScript) by editing the file. But I don't want to have to edit each default file as Salesforce creates them.
Urgency
Moderate
- Langage dominant
- TypeScript
- Étoiles
- 241
- Forks
- 52
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de forcedotcom/code-analyzer
-
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
forcedotcom/code-analyzer#2106 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
forcedotcom/code-analyzer#2094 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
forcedotcom/code-analyzer#2093 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
forcedotcom/code-analyzer#2091 ·
-
[BUG][code-analyzer] sfge: calling toString() on an enum aborts the entry point (UnimplementedMethodException)Peut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
forcedotcom/code-analyzer#2090 ·
Toutes les issues de forcedotcom/code-analyzer
Issues similaires
-
[Bug]: Pen taps on sidebar rows start a drag and lose their clickPeut-être pris @makhnatkin l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
b310-digital/teammapper#1515 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100
puckeditor/puck#1884 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
outline/outline#14378 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
nicobailon/pi-web-access#642 ·
Les mainteneurs répondent en général sous 1 jour