dev-infra: linting fails for `BUILD.bazel` file and `ng-dev format files` does not fix the failure

Aperta

#2,346 aperta il 3/2/2021 0 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@josephperrott ci sta già lavorando.

Dal 3/2/2021.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

area: build & ci P3

In angular/angular#40621, we run into a situation where the ng-dev format ... --check command (run as part of the lint job on CI) finds some formatting issues with a BAZEL.build file and fails. It suggests (or prompts the user) to run ng-dev format files ... to fix the issue, but that command does not fix the linting failure.

In that case, the problem was a glob pattern that included a constant string (see https://github.com/angular/angular/pull/40621#issuecomment-772482472). The fix was to manually change the pattern as follows:

-srcs = glob(["constant.string", "glob/*.pattern"]),
+srcs = ["constant.string"] + glob(["glob/*.pattern"]),

Not sure what is the best way to fix this. Ideally, we should suggest (or prompt the user) to run a command that actually fixes the error. If there is no such command, we should at least show the error message, so users can figure out how to manually fix it themselves.

Lingua principale: JavaScript
Stelle: 77
Fork: 84
Merge medio: 2g 9h
PR unite (30g): 42

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

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

Altre issue di angular/dev-infra

Support code splitting for `app_bundle` rule Aperta

Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100

angular/dev-infra#949 · Aperta il 14/12/2022
Npm version conflicts when lts versions are tagged Aperta

Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100

angular/dev-infra#925 · Aperta il 8/12/2022
Consider supporting automatic conflict resolution for e.g. Yarn Aperta

Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100

angular/dev-infra#760 · 1 commento · Aperta il 5/8/2022
Document unpublish options for a botched release Aperta

domain: release automation

angular/dev-infra#746 · 1 assegnatario · Aperta il 2/8/2022
Handle RC cut failure Aperta

domain: release automation

angular/dev-infra#745 · 1 assegnatario · Aperta il 2/8/2022

Tutte le issue di angular/dev-infra

Issue simili

Ajv defaults: non-standard "format": "json" in third-party schemas (e.g. Notion) warns twice per property — register it like #2601 Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100

modelcontextprotocol/typescript-sdk#2854 · Aperta il 24/9/2026
[Bug]: ctx_search rejects singular 'query' param despite schema declaring it valid Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

mksglu/context-mode#1200 · Aperta il 24/9/2026
Duplicated 'Connect to Database Using C#' link in Postgres Tutorial Page Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

neondatabase/website#5944 · Aperta il 24/9/2026
bbb-conf --set-port-range writes port values as YAML strings, so mediasoup ignores the configured range Aperta

module: core

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

bigbluebutton/bigbluebutton#25849 · Aperta il 24/9/2026
[chore]: test(search): Migrate ResultItemTitle unit tests to TypeScript Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

jaegertracing/jaeger-ui#4506 · Aperta il 24/9/2026

Altre issue su JavaScript