Skip to content

Preserve case for --version argument in core download (when it makes sense)#3283

Merged
danielbachhuber merged 2 commits into
wp-cli:masterfrom
jorgeatorres:master
Aug 12, 2016
Merged

Preserve case for --version argument in core download (when it makes sense)#3283
danielbachhuber merged 2 commits into
wp-cli:masterfrom
jorgeatorres:master

Conversation

@jorgeatorres
Copy link
Copy Markdown

Fixes #3282.

I know this is something you can easily patch yourselves, but I wanted to submit the PR in case that helps. Let me know if there's something I need to adjust before you can (maybe) merge this 😸.

@danielbachhuber
Copy link
Copy Markdown
Member

Let me know if there's something I need to adjust before you can (maybe) merge this

Can you include functional tests please?

@jorgeatorres
Copy link
Copy Markdown
Author

Sure. I'm on it.

@jorgeatorres
Copy link
Copy Markdown
Author

@danielbachhuber: added. Please let me know if everything checks out.

@danielbachhuber
Copy link
Copy Markdown
Member

👍

@danielbachhuber danielbachhuber merged commit 785c971 into wp-cli:master Aug 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants