open-telemetry/opentelemetry-specification

Replace Python yamllint with NPM yaml-lint

Offen

#2.347 geöffnet am 15.02.2022

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (3 Reaktionen) (1 zugewiesene Person)Makefile (987 Forks)auto 404
Stalehelp wantedspec:miscellaneous

Repository-Metriken

Stars
 (4.267 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 10T 15h) (43 gemergte PRs in 30 T)

Beschreibung

Most of the linters are in Node, so changing yamllint with Node version simplifies local testing.

Downside: I don't know what feature-parity is between the linters.

$ npm install yaml-lint
$ npx yamllint '**/*.yaml' --ignore='node_modules/**'
✔ YAML Lint successful.

(Note: this depends on #2323)

Contributor Guide

Research-Richtung
Installiere npm yaml lint und aktualisiere die Linter Konfiguration, um das Python yamllint zu ersetzen. Beachte, dass dies von Issue #2323 abhängt, stelle also sicher, dass diese zuerst gelöst ist. Überprüfe, ob alle YAML Dateien den Lint mit dem neuen Tool bestehen.
Tech Stack
pythonnodejs
Domain
developer experiencetooling
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitNPM
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80