Skip to content

Add --woocommerce flag to scaffold _s command#159

Merged
schlessera merged 3 commits intowp-cli:masterfrom
CodeProKid:feature/139
Jul 20, 2018
Merged

Add --woocommerce flag to scaffold _s command#159
schlessera merged 3 commits intowp-cli:masterfrom
CodeProKid:feature/139

Conversation

@CodeProKid
Copy link
Contributor

Added the new --woocommerce flag to the scaffold _s command.

First time writing behat tests, so hopefully I got them right. The only test I wrote was basically to assert two of the WooCommerce files existed. Not sure if the tests should be more in depth than that, or if there should also be a test to assert the WooCommerce files don't exist when the flag isn't present.

Copy link
Member

@schlessera schlessera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, @CodeProKid, only a minor code style change needed.

* : Include stylesheets as SASS.
*
* [--woocommerce]
* : Include WooCommerce boilerplate files
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CS: End comments with a period ..

@CodeProKid
Copy link
Contributor Author

Thanks @schlessera, this has been fixed 👍

@schlessera schlessera added this to the 1.1.4 milestone Jul 20, 2018
@schlessera schlessera merged commit e5cb0a8 into wp-cli:master Jul 20, 2018
@schlessera schlessera changed the title fixes #139 adding --woocommerce flag to scaffold _s command Add --woocommerce flag to scaffold _s command Jul 20, 2018
danielbachhuber pushed a commit that referenced this pull request Nov 18, 2022
fixes #139 adding --woocommerce flag to scaffold _s command
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