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

Jump to bottom

Add --config-file argument to process custom configuration files #104

Merged
merged 15 commits into from Jan 6, 2022
Merged

Add --config-file argument to process custom configuration files #104

merged 15 commits into from Jan 6, 2022
+1,058 −110

Conversation

@pmbaldha
Copy link
Contributor

@pmbaldha pmbaldha commented Mar 16, 2020 •
edited by schlessera

Fixes wp-cli/ideas#138
@pmbaldha pmbaldha requested a review from as a code owner Mar 16, 2020
@pmbaldha pmbaldha mentioned this pull request Mar 16, 2020
Closed
1 task
@schlessera
Copy link
Member

@schlessera schlessera commented Mar 17, 2020

@pmbaldha Can you add Behat tests to this PR?

@pmbaldha
Copy link
Contributor Author

@pmbaldha pmbaldha commented Mar 30, 2020

@schlessera I have written all related behat tests.

@pmbaldha pmbaldha closed this Mar 30, 2020
@pmbaldha pmbaldha reopened this Mar 30, 2020
schlessera
schlessera requested changes Apr 1, 2020
View changes
features/config-delete.feature Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
src/Config_Command.php Outdated Show resolved Hide resolved
schlessera
schlessera requested changes Apr 1, 2020
View changes
Copy link
Member

@schlessera schlessera left a comment

Code is mostly good, just needs a refactor for get_config_path() method. Most of the other issues are about whitespace.

@pmbaldha pmbaldha requested a review from schlessera Apr 2, 2020
@schlessera schlessera changed the title Add --config-file argument - Custom wp-config.php file parameter for wp config Add --config-file argument to process custom configuration files Apr 8, 2020
@schlessera schlessera force-pushed the 20200312_PB_--config-file_param_introduced branch from 79cd5b4 to 352ea08 Dec 24, 2021
@schlessera schlessera force-pushed the 20200312_PB_--config-file_param_introduced branch from 352ea08 to 0770824 Dec 24, 2021
@schlessera schlessera force-pushed the 20200312_PB_--config-file_param_introduced branch from bb51322 to 5da390f Jan 6, 2022
@schlessera schlessera added this to the 2.1.2 milestone Jan 6, 2022
@schlessera schlessera force-pushed the 20200312_PB_--config-file_param_introduced branch from 5da390f to b3f05ca Jan 6, 2022
@schlessera schlessera merged commit 1e39424 into wp-cli:master Jan 6, 2022
26 checks passed
schlessera added a commit that referenced this issue Jan 6, 2022
@schlessera
Merge pull request #104 from pmbaldha/20200312_PB_--config-file_param…
02926ad 
…_introduced
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@schlessera schlessera

Assignees
No one assigned
Labels
command:config
Projects
None yet
Milestone
2.1.2
Linked issues

Successfully merging this pull request may close these issues.

Custom wp-config.php file parameter for wp config set command
2 participants
@pmbaldha @schlessera