eclipse-score / eclipse-score/devcontainer

relax run-tool

Offen
#148 1 Kommentar 0 Reaktionen 1 zugewiesene Person Beansprucht von @naveena456 Auf GitHub ansehen
good first issue
Vorherrschende Sprache
Shell
Sterne
5
Forks
9
Ø Merge
1 T. 3 Std.
Gemergte PRs (30 T.)
2

Beschreibung

When executed without a devcontainer `run-tool` will run it via bazel, even if the tool is installed locally. Thats a waste of time, considering how slow bazel is.

At the time of writing this https://github.com/eclipse-score/devcontainer/tree/main/tools lists `pre-commit` with version `4.5.1`. So If I have that already installed, there is no reason to run uv via bazel.

Suggested approach:
* if the tool is available, query its version. Beware that version flags are inconsistent across tools
* if the version matches exactly, use the tool directly. Future enhancement: fuzzy match, e.g. same major or same major+minor.
* add a --strict flag to the script, which will not use local tools

Note: this is labeled `good first issue` not because its trivial, but because it has a very clear scope.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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