← Back to issue list

Support for a hidden snapcraft.yaml

View original Github issue

Metadata

Project
snapcraft
Number
#582
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged. Adds support for a hidden .snapcraft.yaml config, erroring if both it and snapcraft.yaml exist. snapcraft init still creates the standard file. Incorporated reviewer terminology feedback and passed retests before merging.

Suggested action:

No scores available.

Issue body

Adding support for .snapcraft.yaml and erroring out if both a .snapcraft.yaml and a snapcraft.yaml are found. `snapcraft init` will still create an unhidden one. LP: #1587933 Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Adds support for a hidden .snapcraft.yaml config, erroring if both it and snapcraft.yaml exist. snapcraft init still creates the standard file. Incorporated reviewer terminology feedback and passed retests before merging.
qwen3.6-35b-a3b-mtp-q6 Merged. Adds .snapcraft.yaml support, erroring if both hidden and unhidden configs exist. snapcraft init still creates the unhidden version. Approved after minor wording tweaks and successful CI retests.
qwen3.6-35b-a3b-mtp-q6 Merged. Adds support for .snapcraft.yaml, errors if both hidden and standard files exist, and keeps snapcraft init creating the standard file. Resolved after minor wording adjustments and CI retests.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1587933 Allow using .snapcraft.yaml as input snapcraft (launchpad) closed Fixed to allow .snapcraft.yaml as an input file. The implementation was verified and the issue closed, enabling fallback to the hidden config when snapcraft.yaml is absent.
76%
#669 Add detection for an existing `.snapcraft.yaml` file before running `snapcraft init` snapcraft merged Merged fix preventing snapcraft init from creating a duplicate snapcraft.yaml when .snapcraft.yaml exists. A unit test was added per reviewer request before merge.
75%
#618 parser - Support .snapcraft.yaml files. snapcraft merged Merged. Adds parser support for .snapcraft.yaml alongside snapcraft.yaml, enforcing that exactly one file exists and erroring otherwise. Successfully integrated into the codebase.
73%
#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.
71%
#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.
71%