Skip to content

fix!: environment variable priority#4544

Merged
lucascolley merged 13 commits intoprefix-dev:mainfrom
Hofer-Julian:fix/env-var-order
Sep 12, 2025
Merged

fix!: environment variable priority#4544
lucascolley merged 13 commits intoprefix-dev:mainfrom
Hofer-Julian:fix/env-var-order

Conversation

@Hofer-Julian
Copy link
Copy Markdown
Contributor

@Hofer-Julian Hofer-Julian commented Sep 9, 2025

Fixes #3801

  • Add actual implementation
  • Ensure that order of pixi shell is also correct
  • User testing

@lucascolley lucascolley added bug Something isn't working breaking Breaks something in the api or config env-vars Related to environment variables labels Sep 9, 2025
@Hofer-Julian Hofer-Julian changed the title fix: environment variable priority fix!: environment variable priority Sep 9, 2025
Hofer-Julian added a commit to Hofer-Julian/rattler that referenced this pull request Sep 9, 2025
@Hofer-Julian Hofer-Julian marked this pull request as ready for review September 10, 2025 13:55
@lucascolley lucascolley self-requested a review September 12, 2025 10:51
Copy link
Copy Markdown
Collaborator

@lucascolley lucascolley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I looked over the tests again and confirmed that they both fail on main!

@lucascolley lucascolley self-requested a review September 12, 2025 11:37
Copy link
Copy Markdown
Collaborator

@lucascolley lucascolley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks Julian, nice work!!

@lucascolley lucascolley merged commit 688ffb1 into prefix-dev:main Sep 12, 2025
41 checks passed
@Hofer-Julian Hofer-Julian deleted the fix/env-var-order branch December 9, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaks something in the api or config bug Something isn't working env-vars Related to environment variables

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(tasks): environment variables broken when defined in surrounding scope

2 participants