PowerShell / PowerShell/PSScriptAnalyzer

Add a test for copyright headers for CI

Aberta
#1,155 0 comentários 1 reação 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Area - Build Issue - Enhancement
Linguagem predominante
C#
Estrelas
2.2k
Forks
414
Merge médio
13h 1min
PRs com merge (30d)
2

Descrição

Found a few files that needed copyright headers for a PR I did with:

Get-ChildItem -Recurse . -Include '*.cs' | ? { $_.Name -notlike '*AssemblyInfo*' } | ? { -not (Select-String -Path $_ -Pattern 'Copyright') }

That worked but still returned some false positives.

Given that there's already documentation tests, we should add a copyright header test too -- but it would require some work, since test assets would need to be moved into directories where they won't be looked for...

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Localize os testes de documentação existentes e os ativos de teste que eles cobrem e, em seguida, revise como o repositório descobre arquivos para as verificações de CI. Use a pesquisa do PowerShell mostrada como contexto para exclusões e falsos positivos. Considera-se concluído quando um teste de cabeçalho de copyright cobre os arquivos pretendidos sem exigir que ativos de teste não relacionados sejam verificados.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
csharp, powershell
Domínio
testing, tooling
Tipo de issue
Funcionalidade
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.