Skip to content

Conversation

nowak-ninja
Copy link

No description provided.

@GrahamCampbell GrahamCampbell changed the title 5.2 fix environment setting [5.2] Fix environment setting Apr 28, 2016
@GrahamCampbell
Copy link
Member

Don't worry about the CS. StyleCI automatically merges the fixes after we merge PRs. Also note that you can download and apply the diff StyleCI gives instead of manually making fixes.

@nowak-ninja
Copy link
Author

nowak-ninja commented Apr 28, 2016

@GrahamCampbell good point, I'm kind of newbie in making PRs here :)

@GrahamCampbell
Copy link
Member

I'm kind of newbie in making PRs here :)

No problem at all. ;)

@taylorotwell
Copy link
Member

What does this actually do / fix?

@nowak-ninja
Copy link
Author

@taylorotwell It fixes ability to run CLI in chosen environment based on .env.* file

@vlakoff
Copy link
Contributor

vlakoff commented Apr 30, 2016

Could $app->detectEnvironment() be called sooner by the user? Because this method writes over $this['env']... Also of course, it would ignore user custom detection.

@taylorotwell
Copy link
Member

What this code does is just not obvious from reading it. Perhaps there is a better solution another way.

@vlakoff
Copy link
Contributor

vlakoff commented May 1, 2016

Ouch, that's a complicated one.

@nowak-ninja nowak-ninja deleted the 5.2_fix_environment_setting branch July 21, 2016 08:23
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.

4 participants