← Back to issue list

meta: parse environment variables (CRAFT-835)

View original Github issue

Metadata

Project
snapcraft
Number
#3645
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged into snapcraft/7.0. Implements parsing for the environment dictionary in snapcraft.yaml at the root and app levels. Approved, passed CI and unit tests, and achieved 100% diff coverage.

Suggested action:

No scores available.

Issue body

- [X] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [X] Have you successfully run `./runtests.sh static`? - [X] Have you successfully run `./runtests.sh tests/unit`? ----- Parse `environment` dictionary in snapcraft.yaml, both for apps and at the root level.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged into snapcraft/7.0. Implements parsing for the environment dictionary in snapcraft.yaml at the root and app levels. Approved, passed CI and unit tests, and achieved 100% diff coverage.
qwen3.6-35b-a3b-mtp-q6 Merged implementation parsing the environment dictionary in snapcraft.yaml for apps and the root level. Review corrected a terminology reference from parts to apps. Changes successfully integrated into the snapcraft/7.0 branch.
qwen3.6-35b-a3b-mtp-q6 Merged into snapcraft/7.0. Adds parsing for the environment dictionary in snapcraft.yaml at the root and app levels. Review clarified scope to exclude parts. Implementation achieved 100% diff coverage and was successfully integrated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2897 meta: include environment in hook wrappers snapcraft merged Merged a fix restoring environment variable inclusion for Snapcraft hook wrappers, correcting a regression that limited it to apps. Approved, passed CI, and modified three files. Resolves LP #1824255.
72%
#4607 fix(application): support snap/craft environment variables snapcraft merged Merged after three approvals and passing CI. The PR adds snap/craft environment variable support across two files. Initial spread test failures and merge conflicts from a concurrent PR were resolved before final integration.
72%