Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

OS env override of nested param in theme does not work #8346

Closed
bep opened this issue Mar 19, 2021 · 1 comment · Fixed by #8349
Closed

OS env override of nested param in theme does not work #8346

bep opened this issue Mar 19, 2021 · 1 comment · Fixed by #8349
Assignees
Milestone

Comments

@bep
Copy link
Member

bep commented Mar 19, 2021

I may be wrong here (because I suspected this to just work).

@bep bep added the Bug label Mar 19, 2021
@bep bep added this to the v0.81.1 milestone Mar 19, 2021
@bep bep self-assigned this Mar 19, 2021
@bep bep changed the title Setting config from env (e.g. HUGO_PARAMS_MYPARAM) does not overwrite params defined in theme modules Setting config from env (e.g. HUGO_PARAMS_MYPARAM)?? Mar 19, 2021
bep added a commit to bep/hugo that referenced this issue Mar 20, 2021
@bep bep changed the title Setting config from env (e.g. HUGO_PARAMS_MYPARAM)?? OS env override of nested param in theme does not work Mar 20, 2021
@bep bep closed this as completed in #8349 Mar 20, 2021
bep added a commit that referenced this issue Mar 21, 2021
The recent fix for #8346 had an unfortunate side-effect, as it prevented overrides of config options used in the module collector, e.g. `HUGO_MODULE_REPLACEMENTS`.

This commit fixes that by applying the overrides twice.
@bep bep modified the milestones: v0.81.1, v0.82 Mar 21, 2021
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 23, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant