Skip to content

/ wp-config-transformer

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

Change to "Happy publishing." per change in core #13

Merged
merged 1 commit into from Jan 12, 2019
Merged

Change to "Happy publishing." per change in core #13

merged 1 commit into from Jan 12, 2019

Conversation

@danielbachhuber
Copy link
Member

danielbachhuber commented Jan 11, 2019

Related wp-cli/config-command#81
@danielbachhuber
Change to "Happy publishing." per change in core
Loading status checks…
4bf7936
@fjarrett
Copy link
Contributor

fjarrett commented Jan 11, 2019

@danielbachhuber Nice catch! Wow, talk about premonition. It just so happens we didn't include that part of the line as the default anchor text for adding configs. 😉
@danielbachhuber
Copy link
Member Author

danielbachhuber commented Jan 11, 2019

Wow, talk about premonition. It just so happens we didn't include that part of the line as the default anchor text for adding configs.

Yep, that was a pretty lucky mistake accident 😄
@danielbachhuber danielbachhuber added this to the 1.2.5 milestone Jan 11, 2019
@danielbachhuber danielbachhuber requested a review from wp-cli/committers Jan 11, 2019
@swissspidy
swissspidy approved these changes Jan 12, 2019
View changes
@schlessera schlessera merged commit 9176b68 into master Jan 12, 2019
2 checks passed
2 checks passed
DEP All dependencies are resolved
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@delete-merged-branch delete-merged-branch bot deleted the happy-publishing branch Jan 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@swissspidy swissspidy

Assignees
No one assigned
Labels
scope:testing
Milestone
1.2.5
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@danielbachhuber @fjarrett @swissspidy @schlessera
You can’t perform that action at this time.