PowerShell / PowerShell/PSScriptAnalyzer

Add a test for copyright headers for CI

Aperta
#1,155 0 commenti 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Area - Build Issue - Enhancement
Lingua principale
C#
Stelle
2.2k
Fork
414
Merge medio
13h 1m
PR unite (30g)
2

Descrizione

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...

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Individua i test della documentazione esistenti e gli asset di test che coprono, quindi esamina il modo in cui il repository individua i file per i controlli CI. Usa la ricerca PowerShell mostrata come contesto per le esclusioni e i falsi positivi. Il lavoro è completato quando un test dell’intestazione di copyright copre i file previsti senza richiedere la scansione di asset di test non correlati.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
csharp, powershell
Ambito
testing, tooling
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.