canonical / canonical/layer-basic

make lint has redundant behavior

Offen
#76 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
10
Forks
42
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Bundletester explicitly calls charm-proof, so we really don't need to call it in the makefile during lint.

background info:

having this added to the makefile is redundant, and necessitates me to fork the makefile in my upper layer if I wish to remove this behavior. Charm-tools hasn't been bumped in quite a while and charms leveraging new features will consistently fail when using keywords like:
- terms
- resources

I feel like having lint fail consistently, when proof has already failed, raises red herrings.

Can we remove the proof step from the lint target to stop the redundancy?

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Untersuche das lint-Ziel des Makefiles und den expliziten charm-proof-Aufruf von Bundletester. Bestätige, dass lint derzeit proof redundant ausführt und dass das Entfernen dieses Aufrufs Bundletester für proof verantwortlich lässt. Erledigt ist die Aufgabe, wenn das lint-Ziel charm-proof nicht mehr aufruft und der bestehende lint-Ablauf intakt bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
build-system
Issue-Typ
Refactoring
Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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