PowerShell / PowerShell/PSScriptAnalyzer

Add a test for copyright headers for CI

Offen
#1,155 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Area - Build Issue - Enhancement
Vorherrschende Sprache
C#
Sterne
2.2k
Forks
414
Ø Merge
13 Std. 1 Min.
Gemergte PRs (30 T.)
2

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Finde die vorhandenen Dokumentationstests und die von ihnen abgedeckten Test-Assets und prüfe anschließend, wie das Repository Dateien für CI-Prüfungen findet. Verwende die gezeigte PowerShell-Suche als Kontext für Ausschlüsse und False Positives. Als erledigt gilt die Aufgabe, wenn ein Copyright-Header-Test die vorgesehenen Dateien abdeckt, ohne dass nicht zugehörige Test-Assets durchsucht werden müssen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp, powershell
Bereich
testing, tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.