fix: reference slug path in for rockcraft.yaml #2
Metadata
Current evaluation
Merged in canonical/rockcraft#1082 via commit cae91c7d, correcting the CLI’s rockcraft.yaml reference link to the documentation slug rockcraft-yaml.
Resolution: closed resolved
Reason: PR #1082 was merged with all required checks passing. Its one-line change fixed the incorrect documentation URL emitted for invalid rockcraft.yaml files.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The existing CLI messages provide the wrong link to the users:
Example:
```
Bad rockcraft.yaml content:
...
For more information, check out: https://documentation.ubuntu.com/rockcraft/latest/reference/rockcraft.yaml
```
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/rockcraft/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [ ] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/rockcraft#1082 via commit cae91c7d, correcting the CLI’s rockcraft.yaml reference link to the documentation slug rockcraft-yaml. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/rockcraft#1082 via commit cae91c7d, correcting the CLI’s rockcraft.yaml documentation link to use the proper slug path. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1082 fixed the CLI's rockcraft.yaml documentation link, resolving the incorrect slug path in commit cae91c7d. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged a one-line fix correcting the CLI's rockcraft.yaml documentation link in commit cae91c7d. | |
| stealth/space-bunny-alpha | — | closed resolved | Corrected the invalid rockcraft.yaml reference URL in CLI error messages; merged as commit cae91c7d. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a one-line fix correcting an incorrect documentation URL in CLI error messages. The change updates the reference path in rockcraft.yaml error prompts to the proper slug path. CI checks passed and reviewers approved before merging. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix correcting an incorrect documentation URL path in CLI error messages for rockcraft.yaml. The update replaces the broken link with the proper reference slug path, ensuring accurate user guidance. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #289 docs: expand rockcraft.yaml reference | rockcraft | merged | Merged into main via commit 654c9c01, reorganizing reference docs and expanding the rockcraft.yaml configuration reference across seven files. | |
| #310 docs: fix url of rockcraft format | rockcraft | merged | Merged in canonical/rockcraft#310 via commit 8145238e, correcting the Rockcraft format URL and resolving the convert-to-pebble-layer.rst linkcheck failure. | |
| #682 docs: fix typo | rockcraft | merged | Merged rockcraft#682 (commit 371bc723) fixing a one-line documentation typo; approved by 3 reviewers, CLA signed, merged. | |
| #2958 fix: update rockcraft documentation link to new url | charmcraft | merged | Merged PR #2958 updating legacy rockcraft documentation links from documentation.ubuntu.com to ubuntu.com/containers/rockcraft/docs across 10 files. | |
| #318 docs: automate CLI reference documentation | rockcraft | merged | Merged PR #318 in commit ca1b1c20, automating Rockcraft's CLI reference documentation and implementing CRAFT-1779. | |
| #1117 fix(cli): refer to rockcraft.yaml in expand-extensions help | rockcraft | closed | Docs-only PR correcting the expand-extensions help text to say rockcraft.yaml. Closed unmerged by the author after maintainer @asanvaq noted it duplicated the work in #1103 and #1110; fix still pending in #1103. | |
| #400 fix(docs): make sure we can build the example rockcraft.yaml | rockcraft | merged | Merged in e7e3091e: corrected the documented example to use armhf and added a spread test verifying that rockcraft.yaml builds. | |
| #1210 fix(commands): reference rockcraft.yaml in extensions help | rockcraft | closed | PR #1210 (fix expand-extensions help to say rockcraft.yaml instead of snapcraft.yaml) was closed unmerged by its author as a duplicate of canonical/rockcraft#1103, which is already approved and CI-passing. | |
| #33 cli: Fix description of "rockcraft pack" | rockcraft | merged | Merged in PR #33 via commit 51918428, correcting “rockcraft pack” help to state that it packs a ROCK rather than a snap. | |
| #1103 fix: remove snapcraft.yaml reference from expand-extensions | rockcraft | open | Two-line help/docstring fix in rockcraft/commands/extensions.py replacing "snapcraft.yaml" with "rockcraft.yaml" in expand-extensions. Approved, CI passing, no unresolved comments; verified still applicable to current main. |