eclipse-score / eclipse-score/devcontainer

relax run-tool

Open
#148 1 comment 0 reactions 1 assignee Claimed by @naveena456 View on GitHub
good first issue
Dominant language
Shell
Stars
5
Forks
9
Avg merge
1d 3h
Merged PRs (30d)
2

Description

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.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.