bazelbuild/bazel

bazel sync --experimental_repository_resolved_file is formatted differently than buildifier

Open

#14.454 geöffnet am 20. Dez. 2021

Auf GitHub ansehen
 (1 Kommentar) (1 Reaktion) (0 zugewiesene Personen)Java (4.465 Forks)batch import
P3help wantednot staleteam-ExternalDepstype: feature request

Repository-Metriken

Stars
 (25.384 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 22T 20h) (77 gemergte PRs in 30 T)

Beschreibung

Description of the problem / feature request:

The output of bazel sync --experimental_repository_resolved_file is formatted differently than buildifier formats .bzl files. If both sync and buildifer are used in a pre-commit hook this can cause the hooks to continuously fail.

Feature requests: what underlying problem are you trying to solve with this feature?

Consistent formatting of bazel files and not having pre-commit hooks continuously fail.

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

Run bazel-sync --experimental_repository_resolved_file=resolved.bzl and then buildifier resolved.bzl and observe that the file has changed in formatting.

What operating system are you running Bazel on?

Debian Linux.

What's the output of bazel info release?

release 4.2.2

Have you found anything relevant by searching the web?

No

Any other information, logs, or outputs that you want to share?

No

Contributor Guide

Research-Richtung
Untersuche, wie Bazel die aufgelöste Datei generiert (wahrscheinlich im Java Code unter src/main/java/com/google/devtools/build/lib/bazel/repository) und vergleiche die Formatierungsregeln mit denen von buildifier. Betrachte die Ausgabe von `bazel sync experimental repository resolved file` und führe buildifier auf derselben Datei aus, um Unterschiede zu identifizieren. Entscheide dann, ob die Bazel Ausgabe angepasst werden soll, um buildifier zu entsprechen, oder ob ein Formatierungsschritt hinzugefügt werden soll. Das Problem könnte auch Änderungen im buildifier Repository erfordern. Beginne damit, den Code zu prüfen, der die aufgelöste Datei schreibt, und sieh nach, ob es einen benutzerdefinierten Formatierer oder eine einfache Ausgabe gibt.
Tech Stack
java
Domain
build systemdeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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
BazelbuildifierStarlark
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60