fix: reference slug path in for rockcraft.yaml #2
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix correcting an invalid documentation URL in CLI error messages for rockcraft.yaml. The maintainer updated the reference path to the correct slug, resolving broken links. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #289 docs: expand rockcraft.yaml reference | rockcraft | merged | Merged after three reviewers approved, the PR reorganized the documentation reference section and expanded the rockcraft.yaml reference guide, modifying 7 files. | |
| #310 docs: fix url of rockcraft format | rockcraft | merged | Merged a documentation fix correcting a broken URL in convert-to-pebble-layer.rst, resolving a linkcheck failure. Approved by two reviewers, passed CI, and modified one file. | |
| #400 fix(docs): make sure we can build the example rockcraft.yaml | rockcraft | merged | Merged to resolve issue #399. Added a spread test validating the example rockcraft.yaml build process for documentation. Approved by three reviewers and passed all CI checks. | |
| #33 cli: Fix description of "rockcraft pack" | rockcraft | merged | Merged a one-line correction to the CLI help text for "rockcraft pack", updating the description to specify it packages a ROCK rather than a snap. Approved by two reviewers and passed CI. | |
| #406 fix(docs): drop "--edge" from rockcraft install instructions | rockcraft | merged | Merged a documentation fix removing the "--edge" flag from rockcraft installation instructions, resolving issue #404. Approved by three reviewers, passed CI, and updated five files. | |
| #634 feat: use versioned form of doc urls | rockcraft | merged | Merged. Updates documentation URLs in error messages and rockcraft init templates to use versioned paths. Approved by two reviewers, passed CI, and modified eight files. | |
| #6 project: add basic rockcraft.yaml parsing and validation (CRAFT-584) | rockcraft | merged | Merged changes adding rockcraft.yaml parsing and validation to halt builds on invalid parts. Renamed the tag field to version and updated the output image suffix to .rock. Approved and passed CI across seven files. | |
| #1210 fix(commands): reference rockcraft.yaml in extensions help | rockcraft | closed | Closed as a duplicate of PR #1103, which already implements the same help text fix and is approved. The change to reference rockcraft.yaml instead of snapcraft.yaml in the expand-extensions command is handled by the superseding pull request. | |
| #1362 docs: Remove outdated report reference from why rockcraft page | rockcraft | merged | Merged documentation changes removing a broken link to a deleted marketing report from the why rockcraft page, resolving issue #1360. Approved by one reviewer, passed all CI checks, and deleted three lines from one file. | |
| #1056 docs: clean up 'rockcraft.yaml' | rockcraft | merged | Merged documentation cleanup for rockcraft.yaml. Updates include upgrading pydantic-kitbash to v0.1.1, removing unnecessary module declarations, and normalizing field order. Approved by two reviewers and merged despite unrelated snap-test failures. |