Skip to content

feat(registry): use ubi backend for func-e#5273

Merged
jdx merged 1 commit into
jdx:mainfrom
risu729:fix-registry
Jun 3, 2025
Merged

feat(registry): use ubi backend for func-e#5273
jdx merged 1 commit into
jdx:mainfrom
risu729:fix-registry

Conversation

@risu729

@risu729 risu729 commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

In #5266, tetratelabs/func-e was added, but it didn't have the backend.
Since aqua-registry doesn't contain it, I guess it was meant to use ubi.
I confirmed it can be installed.

@github-actions

github-actions Bot commented Jun 3, 2025

Copy link
Copy Markdown

If you are adding a new tool to mise, note the following:

  • New asdf plugins are not accepted
  • Tools may be rejected if they are not notable
  • A test is required in registry.toml

@jdx jdx merged commit a3c8550 into jdx:main Jun 3, 2025
29 checks passed
@risu729 risu729 deleted the fix-registry branch June 3, 2025 12:22
jdx pushed a commit that referenced this pull request Jun 4, 2025
My apologies, #5266 didn't work, so this
is the follow-up.

I confirmed all links (except for 1password) work with `mise exec lychee
-- lychee docs/.vitepress/dist/registry.html` (with
#5271 and
#5273).

- Since `aqua:1password/cli` uses `http` type, we should link to
https://developer.1password.com/ I guess, but didn't know where to
declare it.
PR added the package:
aquaproj/aqua-registry#26217

https://github.com/aquaproj/aqua-registry/blob/main/pkgs/1password/cli/registry.yaml

- I switched to `kubernetes/kubernetes/kubectl-convert` from its alias
`kubernetes/kubectl-convert`, to build the correct url.

https://github.com/aquaproj/aqua-registry/blob/main/pkgs/kubernetes/kubernetes/kubectl-convert/registry.yaml

- `dotnet`, `gem`, `spm` backends are not used in the registry
currently, so I ignored them.
I confirmed the page can be rendered even if `url` is an empty string.

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
jdx pushed a commit that referenced this pull request Jun 9, 2025
### 🚀 Features

- **(aqua)** support no_asset and error_message by
[@risu729](https://github.com/risu729) in
[#5303](#5303)
- **(registry)** use ubi backend for func-e by
[@risu729](https://github.com/risu729) in
[#5273](#5273)

### 🐛 Bug Fixes

- **(task)** use empty string for the default value of option by
[@risu729](https://github.com/risu729) in
[#5309](#5309)

### 📚 Documentation

- **(registry)** fix links of registry by
[@risu729](https://github.com/risu729) in
[#5266](#5266)
- **(registry)** fix links to tools by
[@risu729](https://github.com/risu729) in
[#5272](#5272)

### 🧪 Testing

- **(registry)** fix test typos by
[@risu729](https://github.com/risu729) in
[#5269](#5269)

### 🛡️ Security

- **(security)** prevent untarring outside expected path by
[@jdx](https://github.com/jdx) in
[#5279](#5279)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants