Rule request: `Get-Content x | ConvertFrom-Json` should suggest adding `-raw` for `Get-content`
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
Nenhum arquivo-fonte ou teste é especificado. Comece revisando as regras existentes do PSScriptAnalyzer para Get-Content canalizado para ConvertFrom-* e a issue vinculada do PowerShell; determine o tratamento pretendido para comentários JSON e o possível caso de XML. Considera-se concluído quando o analisador sugere -Raw de forma consistente quando apropriado e o comportamento está coberto por testes.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Summary of the new feature
When you call get-content file.json | convertfrom-json, funny things happen when file.json contains a comment at the top of the file.
Multiple places suggest that it's better to use get-content -raw for performance and predictability.
See:
- https://github.com/PowerShell/PowerShell/issues/12229
- https://stackoverflow.com/questions/71025659/set-property-value-on-object-loaded-from-json-containing-comments
Proposed technical implementation details (optional)
When get-content is piped to convertfrom-*, PSScriptAnalyzer should suggest adding -raw to get-content.
The same could apply to [xml] get-content file.xml.
What is the latest version of PSScriptAnalyzer at the point of writing
1.21
- Linguagem predominante
- C#
- Estrelas
- 2.2k
- Forks
- 415
- Merge médio
- 13h 1min
- PRs com merge (30d)
- 2
Preparar o ambiente
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 PowerShell/PSScriptAnalyzer
-
Up-for-Grabs
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 78/100
PowerShell/PSScriptAnalyzer#2213 · 2 comentários ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
PowerShell/PSScriptAnalyzer#2219 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
PowerShell/PSScriptAnalyzer#2218 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
PowerShell/PSScriptAnalyzer#2217 · 1 comentário ·
-
PSUseConsistentIndentation double-indents attribute bodies that open a scriptblock (`[Attr({ … })]`)Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 70/100
PowerShell/PSScriptAnalyzer#2216 · 2 comentários · 1 reação ·
Todas as issues de PowerShell/PSScriptAnalyzer
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
SubtitleEdit/subtitleedit#15462 ·
Mantenedores costumam responder em até 1 dia
-
:watch: Not Triaged
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
comp:instrumentation.aspnetcore
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
open-telemetry/opentelemetry-dotnet-contrib#5427 ·
Mantenedores costumam responder em até 1 dia
-
design-proposal
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
dotnet/aspnetcore#69592 ·
Mantenedores costumam responder em até 1 dia
-
Client Container Registry customer-reported needs-team-attention question Service Attention
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
Azure/azure-sdk-for-net#63470 · 3 comentários · 1 reação ·
Mantenedores costumam responder em até 1 dia