-
Notifications
You must be signed in to change notification settings - Fork 92
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
Add testing suite, initial tests, and CircleCI #503
Conversation
@@ -0,0 +1,152 @@ | |||
#!/usr/bin/env bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boilerplate from https://developer.wordpress.org/cli/commands/scaffold/plugin-tests/
@@ -0,0 +1,47 @@ | |||
machine: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boilerplate from https://developer.wordpress.org/cli/commands/scaffold/plugin-tests/
@@ -0,0 +1,31 @@ | |||
<?php |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boilerplate from https://developer.wordpress.org/cli/commands/scaffold/plugin-tests/
eae4461
to
e204911
Compare
ce7aef3
to
8bce31f
Compare
So this is cool, I took the liberty of fixing the phpcs CLI arguments. Although now you have 1000+ errors 😆 to fix, so you should do a sniffer patch sometime that addresses all of this. |
Thanks @cocojoe! It occurs to me that I didn't want to run the pull |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice to see some tests 👍
78caf13
to
4b58c9a
Compare
wp scaffold plugin-tests auth0 --ci=circle
to add suitetestWPAuth0Options