Skip to content

docs(registry): fix links of registry#5266

Merged
jdx merged 4 commits into
jdx:mainfrom
risu729:fix-registry-urls
Jun 3, 2025
Merged

docs(registry): fix links of registry#5266
jdx merged 4 commits into
jdx:mainfrom
risu729:fix-registry-urls

Conversation

@risu729

@risu729 risu729 commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

Fixes #5265.

aqua:pipe-cd/pipecd/pipectl linked to https://github.com/pipe-cd/pipecd/pipectl, so fixed it to only use the repo name instead of the full aqua package name: https://github.com/pipe-cd/pipecd
(This doesn't support aqua packages starting with gitlab.com, but there are no such packages in the registry now.)

Also, asdf:https://gitlab.com/td7x/asdf/adr-tools pointed https://github.com/https, so use slug as is if it's a url (starting with http).

@risu729 risu729 force-pushed the fix-registry-urls branch from 6608f1c to 3a34557 Compare June 2, 2025 21:29
@risu729 risu729 force-pushed the fix-registry-urls branch from 9b0294e to 03360da Compare June 2, 2025 21:44
@risu729 risu729 marked this pull request as draft June 2, 2025 21:47
@risu729 risu729 marked this pull request as ready for review June 3, 2025 07:56
@jdx jdx merged commit 61aafab into jdx:main Jun 3, 2025
18 checks passed
jdx pushed a commit that referenced this pull request Jun 3, 2025
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.
@risu729 risu729 deleted the fix-registry-urls 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