Revisit the use of removeNPMAbsolutePaths
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
- Type d'issue
- Refactorisation
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- typescript
Piste de recherche
Commencez par localiser removeNPMAbsolutePaths et examinez comment il traite les fichiers package.json dans node_modules. Reproduisez npm ci avec le package-lock.json actuel et comparez le diff obtenu, y compris les modifications du saut de ligne final. C’est terminé lorsqu’il est déterminé si le nettoyage est toujours nécessaire dans CI et que les modifications contradictoires décrites ici sont évitées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
It appears that
npm cicreates a huge diff. None of the package.json files in node_modules have a final newline. Is this something that should be done separate from this PR?
Originally posted by @RA80533 in https://github.com/github/codeql-action/issues/487#issuecomment-838000019
- The current package-lock.json file was generated as a v2 lock file by npm 7.x. Major releases of npm prior to 7.x inserted an extra property in package.json files residing in the node_modules folder for which removeNPMAbsolutePaths was used.
In other words, removeNPMAbsolutePaths no longer has anything to clean up.EDIT: This might not be true for the CI. This specific detail needs to be investigated. - removeNPMAbsolutePaths has the unfortunate side effect of stripping the final newline from any file it touches. npm attempts to insert a final newline if it finds that it's missing. The two work against each other to a disastrous effect. VS Code's version control integration has to be reloaded if either of them tries to undo the other because of the enormous amount of changes they make.
- Langage dominant
- TypeScript
- Étoiles
- 1.6k
- Forks
- 493
- Merge moyen
- 1 j 13 h
- PR mergées (30 j)
- 44
Guide de contribution
Ouvrir 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 github/codeql-action
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
github/codeql-action#4052 · 4 commentaires ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
github/codeql-action#4078 · 1 commentaire ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
github/codeql-action#4008 · 9 commentaires ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 52/100
github/codeql-action#3978 · 4 commentaires · 1 réaction ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
github/codeql-action#3915 · 6 commentaires · 3 réactions ·
Toutes les issues de github/codeql-action
Issues similaires
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
-
Mend: dependency security vulnerability untriaged
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100