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

Existing variable takes precendence when performing substitution #313

Merged
merged 1 commit into from
Apr 13, 2018
Merged

Existing variable takes precendence when performing substitution #313

merged 1 commit into from
Apr 13, 2018

Conversation

jcechace
Copy link
Contributor

See issue #312

@stale
Copy link

stale bot commented Nov 26, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Nov 26, 2017
@jcechace
Copy link
Contributor Author

Is this project actively maintained?

@stale stale bot removed the wontfix label Nov 26, 2017
@stale
Copy link

stale bot commented Jan 25, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jan 25, 2018
@siegy22
Copy link
Contributor

siegy22 commented Jan 26, 2018

Hey @jcechace..

@bkeepers can't maintain this project because he doesn't have the time to do so 😐

@stale stale bot removed the wontfix label Jan 26, 2018
@stale
Copy link

stale bot commented Mar 27, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Mar 27, 2018
@stale stale bot closed this Apr 3, 2018
@bkeepers bkeepers mentioned this pull request Apr 10, 2018
Closed
9 tasks
@cadwallion cadwallion self-assigned this Apr 11, 2018
@cadwallion
Copy link
Collaborator

👋 @jcechace !

There's a new trio of maintainers for dotenv as of this week, as mentioned in #326 . I'll be reviewing this PR and the issue it came from shortly. Thank you for this contribution, and am sorry this PR languished as long as it did!

@cadwallion cadwallion reopened this Apr 11, 2018
@cadwallion
Copy link
Collaborator

This makes sense, merging in. Thank you for the contribution!

@cadwallion cadwallion merged commit eb0cf3b into bkeepers:master Apr 13, 2018
@cadwallion cadwallion mentioned this pull request Apr 13, 2018
Closed
@jonmagic
Copy link
Collaborator

We'll be cutting a new release next week that will include the changes from this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cadwallion cadwallion

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jcechace @siegy22 @cadwallion @jonmagic