Skip to content

registry: add SonarQube CLI (aqua:SonarSource/sonarqube-cli)#9824

Merged
jdx merged 1 commit into
jdx:mainfrom
3PeatVR:3PeatVR/registry-add-SonarQubeCLI
May 15, 2026
Merged

registry: add SonarQube CLI (aqua:SonarSource/sonarqube-cli)#9824
jdx merged 1 commit into
jdx:mainfrom
3PeatVR:3PeatVR/registry-add-SonarQubeCLI

Conversation

@3PeatVR

@3PeatVR 3PeatVR commented May 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Adding SonarQube CLI to mise using aqua

Popularity

  • GIthub (SonarSource/sonarqube-cli): 140 stars, 5 forks, last release 0.12.0.1512 on 2026-05-11
  • Aqua registry: already tracked (pkgs/SonarSource/sonarqube-cli/pkg.yaml)
  • CLI for SonarQube, one of the most used static analysis tool

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new registry configuration for the SonarQube CLI. The feedback suggests improving the file's consistency by using 2-space indentation and adding a trailing comma in the backends array. Additionally, the reviewer recommended simplifying the version matching logic in the test command and including the expected 'sonar version' prefix to ensure more robust testing.

Comment thread registry/sonarqube-cli.toml Outdated
Comment thread registry/sonarqube-cli.toml Outdated
@greptile-apps

greptile-apps Bot commented May 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR registers the SonarQube CLI (sonar) as a new mise tool backed by aqua:SonarSource/sonarqube-cli.

  • The single TOML entry is structurally correct: it uses the preferred aqua: backend tier and includes a well-formed test command with a template that correctly trims the 4-part aqua version tag (0.12.0.1512) down to the 3-part string the CLI reports (0.12.0).
  • The adoption threshold documented in CLAUDE.md ("GitHub stars in the thousands, not hundreds") is not met by the numbers in the PR description (140 stars / 5 forks), and the tool is currently in Beta according to SonarSource's own docs — concerns that have already been raised in earlier review threads.

Confidence Score: 4/5

The TOML file itself is mechanically correct and safe to merge from a technical standpoint; the open question is whether the tool meets the registry's popularity bar.

The change is a single, well-formed registry entry with no logic errors. The only risk lies in whether the maintainer will accept it given the tool's current adoption numbers and Beta status — not in the code itself.

registry/sonarqube-cli.toml — maintainer acceptance depends on the tool's popularity, not on anything in the file's content.

Important Files Changed

Filename Overview
registry/sonarqube-cli.toml New registry entry for sonarqube-cli using the preferred aqua backend; structurally well-formed with a correct version-slicing template for the 4-part aqua tag vs 3-part CLI output

Reviews (5): Last reviewed commit: "registry: add SonarQube CLI (aqua:SonarS..." | Re-trigger Greptile

Comment thread registry/sonarqube-cli.toml Outdated
Comment thread registry/sonarqube-cli.toml Outdated
Comment thread registry/sonarqube-cli.toml Outdated
@3PeatVR 3PeatVR force-pushed the 3PeatVR/registry-add-SonarQubeCLI branch from 88d8b19 to 5eacd23 Compare May 13, 2026 03:11
Comment thread registry/sonarqube-cli.toml Outdated
@3PeatVR 3PeatVR force-pushed the 3PeatVR/registry-add-SonarQubeCLI branch 2 times, most recently from d6e42b2 to ef4a7eb Compare May 14, 2026 01:02
@3PeatVR 3PeatVR marked this pull request as ready for review May 14, 2026 01:03
Comment thread registry/sonarqube-cli.toml Outdated
@risu729

risu729 commented May 14, 2026

Copy link
Copy Markdown
Contributor

Why do they have .1512 or like that?

$ mise ls-remote aqua:SonarSource/sonarqube-cli
0.5.0.409
0.6.0.579
0.6.1.603
0.7.0.711
0.8.0.783
0.8.1.798
0.9.0.977
0.10.0.1266
0.11.0.1439
0.12.0.1512

@3PeatVR

3PeatVR commented May 14, 2026

Copy link
Copy Markdown
Contributor Author

@risu729 This represents the build number, and more precisely it increments everytime the CI pipeline runs. This can help finding the number of the CI run in which the release was built 👍 , like "This artifact was built in CI #1512" for instance. Since it is not useful for the user, I guess that is why they do not display it when you do sonar --version.

@risu729

risu729 commented May 14, 2026

Copy link
Copy Markdown
Contributor

I see, that's fair.

@3PeatVR 3PeatVR force-pushed the 3PeatVR/registry-add-SonarQubeCLI branch from ef4a7eb to 945472e Compare May 15, 2026 01:17
@3PeatVR 3PeatVR force-pushed the 3PeatVR/registry-add-SonarQubeCLI branch from 120de41 to f237376 Compare May 15, 2026 01:20
@jdx jdx merged commit 5b89d31 into jdx:main May 15, 2026
33 checks passed
mise-en-dev added a commit that referenced this pull request May 15, 2026
### 🚀 Features

- **(config)** add shell to watch_files run by @risu729 in
[#9810](#9810)
- **(spm)** add artifact bundle support by @ikesyo in
[#9825](#9825)

### 🐛 Bug Fixes

- **(aqua)** reject registry-invalid latest tags by @risu729 in
[#9834](#9834)
- **(patrons)** point sponsor link to https://en.dev by @jdx in
[#9868](#9868)
- **(vfox)** respect default inline shell in cmd.exec by @risu729 in
[#9837](#9837)
- github oauth device flow paths by @jasisk in
[#9791](#9791)

### 📚 Documentation

- Update Walkthrough guide by @thernstig in
[#9853](#9853)

### ⚡ Performance

- **(config)** skip tera render for plain strings by @risu729 in
[#9833](#9833)

### 📦️ Dependency Updates

- update ghcr.io/jdx/mise:rpm docker digest to d2471f2 by @renovate[bot]
in [#9879](#9879)
- update rust docker digest to 5b1e348 by @renovate[bot] in
[#9880](#9880)
- update ghcr.io/jdx/mise:deb docker digest to 0cde829 by @renovate[bot]
in [#9878](#9878)
- update ubuntu docker tag to resolute-20260421 by @renovate[bot] in
[#9881](#9881)
- update ghcr.io/jdx/mise:alpine docker digest to 2d0ea74 by
@renovate[bot] in [#9877](#9877)
- update rust crate phf to 0.13 by @renovate[bot] in
[#9884](#9884)
- update rust crate phf_codegen to 0.13 by @renovate[bot] in
[#9883](#9883)

### 📦 Registry

- use aqua backend for npm by @risu729 in
[#9762](#9762)
- add aqua for buck2 prereleases by @risu729 in
[#9805](#9805)
- add SonarQube CLI
([aqua:SonarSource/sonarqube-cli](https://github.com/SonarSource/sonarqube-cli))
by @3PeatVR in [#9824](#9824)

### New Contributors

- @3PeatVR made their first contribution in
[#9824](#9824)
- @ikesyo made their first contribution in
[#9825](#9825)
- @thernstig made their first contribution in
[#9853](#9853)

## 📦 Aqua Registry Updates

### New Packages (4)

- [`SurgeDM/Surge`](https://github.com/SurgeDM/Surge)
- [`roie/ovw`](https://github.com/roie/ovw)
- [`so-dang-cool/sigi`](https://github.com/so-dang-cool/sigi)
- [`vjeantet/alerter`](https://github.com/vjeantet/alerter)

### Updated Packages (2)

-
[`alltuner/mise-completions-sync`](https://github.com/alltuner/mise-completions-sync)
-
[`str4d/age-plugin-yubikey`](https://github.com/str4d/age-plugin-yubikey)
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.

3 participants