cli: snapcraft init with a base
Metadata
Current evaluation
Merged. Adds a syntax documentation link after snapcraft init execution. Approved by one reviewer, passed CI checks, modified five files, and achieved 100% diff coverage. Resolves LP #1803776.
Suggested action: —
No scores available.
Issue body
Add a link to where the syntax is defined after doing an init.
LP: #1803776
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds a syntax documentation link after snapcraft init execution. Approved by one reviewer, passed CI checks, modified five files, and achieved 100% diff coverage. Resolves LP #1803776. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes adding a syntax documentation link after running snapcraft init. Improves CLI usability by directing users to reference materials. Achieved 100% diff coverage and passed all checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. The pull request adds a link to the base syntax documentation after running snapcraft init, resolving Launchpad bug #1803776. Codecov reported 100% diff coverage, and the maintainer approved the change. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2954 cli: enable config file for base snapcraft command | snapcraft | merged | Merged a single-line update enabling config file parsing for the base snapcraft command. The change extends existing lifecycle configuration support to the main entry point, approved by maintainers and passing all CI checks. | |
| #2285 snapcraft snap: vendor legacy snapcraft | snapcraft | merged | Merged. Vendored legacy snapcraft into the snapcraft snap to resolve LP: #1793261, maintaining support when no base is specified in snapcraft.yaml. Approved by one reviewer and passed CI checks prior to integration. | |
| #5388 fix: don't fail when checking the base | snapcraft | merged | Merged a fix preventing snapcraft from failing early during base checks. Enables user-friendly error reporting for incorrect bases. Approved by two reviewers and merged with 10 additions and 30 deletions across three files. | |
| #146 Syntax of snapcraft.yaml (1.x) | snapcraft | merged | Merged after rebasing to resolve diff conflicts and squashing commits. The pull request updates snapcraft.yaml (1.x) syntax and received final approval before integration. | |
| #2499 many: support for "base: core" in snapcraft.yaml | snapcraft | merged | Added support for base: core in snapcraft.yaml. Merged after approval and passing CI checks. Modified 43 files with a net change of +224/-75 lines. | |
| #5664 fix: snapcraft init crash when snapcraft.yaml exists but is empty | snapcraft | merged | Merged a fix for a snapcraft init crash when snapcraft.yaml exists but is empty. The PR addressed review comments, added unit tests, and passed all CI checks before merging. | |
| #3631 parts: add early base parsing (CRAFT-766) | snapcraft | merged | Merged into snapcraft/7.0. Adds early base parsing for lifecycle commands, verifying the base entry via project YAML and falling back to legacy handlers if not core22. Approved, passed CI, +0.81% coverage. | |
| #776 Updated the contents of the snapcraft init template | snapcraft | merged | Merged update to the snapcraft init template. The maintainer modified 44 lines across three files to align with LP #1620428. Changes were integrated after a retest request, with no blocking CI failures or unresolved review comments. | |
| #4118 init: use core22 base for `snapcraft init` | snapcraft | merged | Merged. Updates snapcraft init to use core22 as the default base instead of core20. The command was relocated from snapcraft_legacy to snapcraft, and a spread test was added. Approved by reviewer and passed CI. | |
| #2309 snap: improve early base detection logic | snapcraft | merged | Merged. Resolved early base detection failure in build providers by updating logic to account for snapcraft's non-project directory execution. Approved by reviewer, passed CI, and merged with minimal changes. |