Skip to content

Change how namespaced options are populated into the env#1287

Merged
jrray merged 1 commit intomainfrom
ns-opt-env-names
Dec 10, 2025
Merged

Change how namespaced options are populated into the env#1287
jrray merged 1 commit intomainfrom
ns-opt-env-names

Conversation

@jrray
Copy link
Collaborator

@jrray jrray commented Oct 23, 2025

Instead of $SPK_OPT_pkgname.optname, which is an invalid environment variable name, use $SPK_PKG_pkgname_OPT_optname. This format then resembles how other environment variables are named, such as $SPK_PKG_pkgname_VERSION.

The old format is also still populated for backward compatibility and to allow projects to update to the new format.

@jrray jrray self-assigned this Oct 23, 2025
@jrray jrray added the enhancement New feature or request label Oct 23, 2025
@codecov
Copy link

codecov bot commented Oct 23, 2025

Codecov Report

❌ Patch coverage is 84.61538% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...spk-schema/crates/foundation/src/option_map/mod.rs 80.00% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@jrray jrray requested a review from rydrman October 23, 2025 21:14
Instead of `$SPK_OPT_pkgname.optname`, which is an invalid environment
variable name, use `$SPK_PKG_pkgname_OPT_optname`. This format then
resembles how other environment variables are named, such as
`$SPK_PKG_pkgname_VERSION`.

The old format is also still populated for backward compatibility and to
allow projects to update to the new format.

Signed-off-by: J Robert Ray <jrray@jrray.org>
@jrray jrray merged commit b3d96b9 into main Dec 10, 2025
9 checks passed
@jrray jrray deleted the ns-opt-env-names branch December 10, 2025 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants