← 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 #3645 into snapcraft/7.0 via 7e7089fb1, adding parsing of root- and app-level environment dictionaries from snapcraft.yaml.

Resolution: closed resolved

Reason: PR #3645 was approved and merged, resolving CRAFT-835. The merge added support for parsing environment dictionaries at both the root and app levels.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged #3645 into snapcraft/7.0 via 7e7089fb1, adding parsing of root- and app-level environment dictionaries from snapcraft.yaml.
stealth/space-bunny-alpha — closed resolved Merged #3645 into snapcraft/7.0 via commit 7e7089fb1, adding parsing of root- and app-level environment dictionaries from snapcraft.yaml.
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.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4607 fix(application): support snap/craft environment variables snapcraft merged Merged PR #4607 as b41ebe6e4, adding SNAP/CRAFT environment-variable support to the application and tests.
72%
#3748 parts: define variables and expand them in snapcraft.yaml snapcraft merged Merged PR #3748 in commit fba20a970, adding global and step-specific SNAPCRAFT_* variables for snapcraft.yaml expansion and lifecycle environments.
71%
#1583259 Snappy needs to influence environment variables in applications snapcraft (launchpad) closed Snapcraft gained top-level and per-app environment keys in snap.yaml, merged in snapcraft#1103 (c66f55656). The review-tool SRU 0.44~16.04.1 was verified and released.
70%