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.

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

Accept true as truthy value in environment vars #645

Closed
wants to merge 1 commit into from
Closed

Accept true as truthy value in environment vars #645

wants to merge 1 commit into from

Conversation

dl1ely
Copy link

@dl1ely dl1ely commented Oct 13, 2017

BOOT_AS_ROOT, BOOT_DISABLE_WATCHERS and BOOT_COLOR environment variables
allow true as a truthy value beside 1 and yes. Fixes #631.

BOOT_AS_ROOT, BOOT_DISABLE_WATCHERS and BOOT_COLOR environment variables
allow true as a truthy value beside 1 and yes. Fixes #631.
Copy link
Member

@martinklepsch martinklepsch left a comment

Choose a reason for hiding this comment

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

I like this change but unfortunately it needs a rebase. Would you mind rebasing Stefan? 🙂

@arichiardi
Copy link
Contributor

Same, rebase and merge 😄

@martinklepsch
Copy link
Member

Rebased and merged: 394a958

@dl1ely dl1ely deleted the feature-631 branch December 21, 2017 20:07
@dl1ely
Copy link
Author

dl1ely commented Dec 21, 2017

Sorry for chiming in that late, but i was very busy. Obviously, picking arbitrary strings and ||ing them together was good enough, @arichiardi. Thanks for merging.

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