Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Add variable expansion #468

Merged
merged 1 commit into from
Apr 24, 2023
Merged

Add variable expansion #468

merged 1 commit into from
Apr 24, 2023

Conversation

invasy
Copy link
Contributor

@invasy invasy commented Apr 24, 2023 •
edited
Loading

@invasy
Add variable expansion (fix joke2k#421)
bee44b9 
- Expand variables referenced as `$VAR` or `${VAR}`.
- Detect infinite recursion in expansion (self-reference).
@invasy invasy mentioned this pull request Apr 24, 2023
Closed
@invasy invasy changed the title Add variable expansion (fix #421) Add variable expansion Apr 24, 2023
@coveralls
Copy link

Coverage Status

Coverage: 92.215% (+0.3%) from 91.921% when pulling bee44b9 on invasy:feature/expansion into 69b4bc9 on joke2k:develop.

@sergeyklay sergeyklay merged commit c61bfb0 into joke2k:develop Apr 24, 2023
@sergeyklay
Copy link
Collaborator

Thank you for the patch, and for helping make django-environ better!

@elmcrest
Copy link

elmcrest commented May 5, 2023

Hey @sergeyklay, is there already a new release available including this change?
on pypi.org I can see 0.10.0 from 3rd of march, so this can't be included. are there plans for a new release?

thx and cheers,
Marius

@sergeyklay
Copy link
Collaborator

Hi, @elmcrest. I'll release new version ASAP.

@sergeyklay sergeyklay self-assigned this Jul 14, 2023
@sergeyklay sergeyklay added the enhancement New feature or request label Jul 14, 2023
@David-Wobrock David-Wobrock mentioned this pull request Sep 5, 2023
Open
@potasiak potasiak mentioned this pull request Nov 11, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sergeyklay sergeyklay sergeyklay approved these changes

Assignees

@sergeyklay sergeyklay

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@invasy @coveralls @sergeyklay @elmcrest