Skip to content

Handle multiple config entries #4

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

Closed
wants to merge 3 commits into from
Closed

Handle multiple config entries #4

wants to merge 3 commits into from

Conversation

travis-ray
Copy link

If app/etc/env.php has multiple config sections with lines matching host, username, password,
or dbName it would fail, combining both entries. This may not be perfect, but it selects the first entry for each, which worked for my case at least.

@MagePsycho
Copy link
Owner

Thanks for your suggestion. Currently has been FIXED with grep -m 1 command itself. We really appreciated your contribution.

@MagePsycho MagePsycho closed this Mar 17, 2021
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.

2 participants