Revisit the use of removeNPMAbsolutePaths
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 25/100
- Tipo de issue
- Refatoração
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Estagnada
- Stack de tecnologia
- typescript
Direção de pesquisa
Comece localizando removeNPMAbsolutePaths e inspecione como ele processa arquivos package.json em node_modules. Reproduza npm ci com o package-lock.json atual e compare o diff resultante, incluindo alterações na quebra de linha final. Considera-se concluído quando for determinado se a limpeza ainda é necessária em CI e forem evitadas as alterações conflitantes descritas aqui.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- TypeScript
- Estrelas
- 1.6k
- Forks
- 493
- Merge médio
- 1d 13h
- PRs com merge (30d)
- 44
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de github/codeql-action
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
github/codeql-action#4052 · 4 comentários ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 45/100
github/codeql-action#4078 · 1 comentário ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
github/codeql-action#4008 · 9 comentários ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 52/100
github/codeql-action#3978 · 4 comentários · 1 reação ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 48/100
github/codeql-action#3915 · 6 comentários · 3 reações ·
Todas as issues de github/codeql-action
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
mksglu/context-mode#1200 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
jaegertracing/jaeger-ui#4506 ·
-
area:desktop area:ui bug platform:macos
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
anthropics/claude-code#96687 ·
-
good first issue
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 95/100
AOSSIE-Org/DebateAI#582 · 2 comentários ·