Skip to content

fix: revert env var changes#4346

Merged
ruben-arts merged 5 commits intoprefix-dev:mainfrom
Hofer-Julian:revert-env-var
Aug 14, 2025
Merged

fix: revert env var changes#4346
ruben-arts merged 5 commits intoprefix-dev:mainfrom
Hofer-Julian:revert-env-var

Conversation

@Hofer-Julian
Copy link
Contributor

@Hofer-Julian Hofer-Julian commented Aug 14, 2025

Unfortunately, we found too many issues with our current env var handling, and @ruben-arts and I couldn't find a clear way to go forward here.

This PR reverts all changes that we added recently.

@magentaqin

@Hofer-Julian Hofer-Julian changed the title Revert env var changes fix: revert env var changes Aug 14, 2025
@ruben-arts
Copy link
Contributor

For me locally this solved all the issues I had, while reintroducing the original issue. But I think this is the best we can do in the short term.

@lucascolley
Copy link
Collaborator

the plan sounds good to me. Let's make sure we keep track of the docs that were added, they can be reintroduced with the new implementation.

@Hofer-Julian
Copy link
Contributor Author

the plan sounds good to me. Let's make sure we keep track of the docs that were added, they can be reintroduced with the new implementation.

Yep, agreed

@Hofer-Julian Hofer-Julian marked this pull request as ready for review August 14, 2025 11:05
@ruben-arts ruben-arts merged commit fb167d8 into prefix-dev:main Aug 14, 2025
78 of 80 checks passed
@Hofer-Julian Hofer-Julian deleted the revert-env-var branch August 14, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants