Skip to content

Comments

config: rework eval order and respect env vars#5672

Merged
neersighted merged 1 commit intopython-poetry:masterfrom
abn:fix-config-eval
May 23, 2022
Merged

config: rework eval order and respect env vars#5672
neersighted merged 1 commit intopython-poetry:masterfrom
abn:fix-config-eval

Conversation

@abn
Copy link
Member

@abn abn commented May 23, 2022

This change ensures that we respect POETRY_CONFIG_DIR as well as
ensure environment variables are respected for POETRY_CACHE_DIR.

This change also fixes an issue with repository cache directories not
respecting base cache directory configuration changes.

Resolves: #2445
Closes: #5301

@abn abn requested a review from a team May 23, 2022 17:28
@abn abn marked this pull request as ready for review May 23, 2022 17:28
This change ensures that we respect `POETRY_CONFIG_DIR` as well as
ensure environment variables are respected for `POETRY_CACHE_DIR`.

This change also fixes an issue with repository cache directories not
respecting base cache directory configuration changes.
@abn abn force-pushed the fix-config-eval branch from f5772a0 to 5231b57 Compare May 23, 2022 17:48
@abn abn requested a review from Secrus May 23, 2022 17:51
@Secrus Secrus requested a review from a team May 23, 2022 18:10
@github-actions
Copy link

This pull request 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 Feb 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

POETRY_CACHE_DIR or cache-dir config are not used at all.

3 participants