-
-
Notifications
You must be signed in to change notification settings - Fork 640
registry: add missing cargo backends #6307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
14a12aa
to
b80ba59
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds missing cargo backend configurations to various tools in the registry. The main purpose is to expand installation options by adding cargo as an additional backend for Rust tools that are published to crates.io.
- Adds cargo backends to approximately 50+ tools that are written in Rust
- Includes explanatory comments for tools where cargo backends are intentionally not added (e.g., ruff, uv, tiny, zola)
- Reformats some backend arrays to multiline format for better readability
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
registry.toml
Outdated
@@ -830,7 +837,7 @@ delta.backends = [ | |||
] | |||
delta.test = ["delta --version", "delta {{version}}"] | |||
deno.description = "A modern runtime for JavaScript and TypeScript (Builtin plugin)" | |||
deno.backends = ["core:deno"] | |||
deno.backends = ["core:deno", "cargo:deno"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we don't need fallbacks for core
backends.
registry.toml
Outdated
@@ -930,6 +938,7 @@ dotenv-linter.description = "Lightning-fast linter for .env files. Written in Ru | |||
dotenv-linter.backends = [ | |||
"aqua:dotenv-linter/dotenv-linter", | |||
"ubi:dotenv-linter/dotenv-linter", | |||
"cargo:dotenv-linter", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reasons for preferring cargo
to asdf
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hadnt noticed any documented preferred order in the backends.
Happy to put cargo after asdf.
registry.toml
Outdated
@@ -2757,6 +2798,7 @@ rocq.aliases = ["coq"] | |||
ruby.description = "Ruby language" | |||
ruby.backends = ["core:ruby"] | |||
ruff.description = "An extremely fast Python linter and code formatter, written in Rust" | |||
# ruff intends to publish to crates.io soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can add cargo:ruff
and comment it out?
registry.toml
Outdated
@@ -3196,6 +3248,7 @@ tilt.description = "Define your dev environment as code. For microservice apps o | |||
tilt.backends = ["aqua:tilt-dev/tilt", "asdf:eaceaser/asdf-tilt"] | |||
timoni.description = "Timoni is a package manager for Kubernetes, powered by CUE and inspired by Helm" | |||
timoni.backends = ["aqua:stefanprodan/timoni", "asdf:Smana/asdf-timoni"] | |||
# tiny is no longer published on crates.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this comment is not necessary. tiny
is not a real tool.
8f1c281
to
5013905
Compare
No description provided.