open-telemetry/opentelemetry-specification

Replace Python yamllint with NPM yaml-lint

Aperta

#2347 aperta il 15 feb 2022

Vedi su GitHubGuida contributor
 (1 commento) (3 reazioni) (1 assegnatario)Makefile (987 fork)auto 404
Stalehelp wantedspec:miscellaneous

Metriche repository

Star
 (4267 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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)

Guida contributor

Direzione di ricerca
Installa npm yaml lint e aggiorna la configurazione del linter per sostituire il Python yamllint. Nota che questo dipende dall'issue #2323, quindi assicurati che sia risolta prima. Verifica che tutti i file YAML passino il linting con il nuovo strumento.
Tech stack
pythonnodejs
Dominio
developer experiencetooling
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNPM
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80