project: snapcraft.yaml in a snap directory
Metadata
Current evaluation
Merged into master. Defaults new projects to place snapcraft.yaml inside a snap directory while retaining support for legacy locations. Approved and passed CI checks.
Suggested action: —
No scores available.
Issue body
Default new projects to have snapcraft.yaml inside a snap directory
without deprecating the other two options.
LP: #1659085
Signed-off-by: Kyle Fazzari <kyle@canonical.com>
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into master. Defaults new projects to place snapcraft.yaml inside a snap directory while retaining support for legacy locations. Approved and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Defaults new projects to store snapcraft.yaml inside a snap/ directory while maintaining backward compatibility with legacy paths. Changes passed code coverage checks and received maintainer approval. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Defaults new projects to store snapcraft.yaml inside a snap/ directory, maintaining support for alternative locations. Changes passed code coverage checks and received maintainer approval. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1601 snapcraft.yaml: don't re-use build dir | snapcraft | merged | Merged a fix for a source-build directory clash in snapcraft.yaml. The build directory was renamed to apt-build to avoid scriptlet conflicts. Approved by reviewers and passed CI, resolving the failure. | |
| #3526 project: validate snapcraft yaml before using it (CRAFT-55) | snapcraft | closed | Merged into master. Moves snapcraft YAML validation to get_project() to prevent errors during extension application and JSON schema checks. Approved by two reviewers. | |
| #725 Support having the snapcraft.yaml in a subdir. | snapcraft | merged | Merged a fix for a recursion crash when snapcraft.yaml resides in a project subdirectory. The change skips the config directory during processing, resolving Launchpad bug #1600428. Modified 7 files with 85 additions and 4 deletions. | |
| #4662 fix(test): do not create 2 snapcraft.yaml and confuse location | snapcraft | merged | Merged a test fix preventing duplicate snapcraft.yaml creation in conflicting root and snap directories. Resolves silent overrides and ensures hooks are detected by clarifying fixture paths. Approved and merged. | |
| #3637 meta: generate basic snap.yaml (CRAFT-801) | snapcraft | merged | Merged into snapcraft/7.0, adding automatic generation of a basic snap.yaml to enable snap packing. Approved by a reviewer, passed CI, and increased test coverage by 1.16%. | |
| #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. | |
| #3608 many: rename snapcraft package to snapcraft_legacy (CRAFT-726) | snapcraft | merged | Merged after renaming the snapcraft package to snapcraft_legacy to make room for a new craft libraries-based application. Approved, passed CI, and modified 557 files. The application entry point remains unchanged and will be handled separately. | |
| #49 packages: add snap package support | craft-parts | merged | Merged after implementing snap download and installation for stage and build snaps. Approved by two reviewers, passed CI, and added 1,072 lines across five files. Follows snapcraft conventions with future refactoring planned. | |
| #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. | |
| #2223 snap: prepare override scripts to allow rebuilding | snapcraft | merged | Merged into master. Updated snapcraft.yaml override scripts to fix ctypes linking and PyYAML patching, enabling easier project rebuilding. Approved by reviewer and passed CI checks. |