-
Notifications
You must be signed in to change notification settings - Fork 191
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 some guidelines for how to test well you code before creating PR #2007
Add some guidelines for how to test well you code before creating PR #2007
Conversation
5d2df36
to
4e880cf
Compare
CONTRIBUTING.md
Outdated
@@ -71,3 +71,12 @@ Follow the steps below to configure ESLint with PHPStorm: | |||
Open any JS file and you'll see suggestions from ESLint, if any. | |||
|
|||
That's all! You're done. | |||
|
|||
## How to test code before creating PR | |||
You can create a awesome PR if: |
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.
"an awesome"
CONTRIBUTING.md
Outdated
|
||
## How to test code before creating PR | ||
You can create a awesome PR if: | ||
1. `debug.log` is empty after testing code functionality. |
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.
The debug.log
found in /wp-content/
CONTRIBUTING.md
Outdated
## How to test code before creating PR | ||
You can create a awesome PR if: | ||
1. `debug.log` is empty after testing code functionality. | ||
2. Do not contain any testing purpose code. |
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.
"Does not contain and testing code like var_dump()
and console.log()
CONTRIBUTING.md
Outdated
You can create a awesome PR if: | ||
1. `debug.log` is empty after testing code functionality. | ||
2. Do not contain any testing purpose code. | ||
3. Passes all phpunit test. |
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.
"tests"
CONTRIBUTING.md
Outdated
1. `debug.log` is empty after testing code functionality. | ||
2. Do not contain any testing purpose code. | ||
3. Passes all phpunit test. | ||
4. No formatting issue while check formatting `php_codesniffer`/`eslint` with `WordPress Coding Standard`. |
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.
"issues when checking formatting via"
CONTRIBUTING.md
Outdated
2. Do not contain any testing purpose code. | ||
3. Passes all phpunit test. | ||
4. No formatting issue while check formatting `php_codesniffer`/`eslint` with `WordPress Coding Standard`. | ||
5. Works fine at least on PHP `5.3` and PHP `7.*` |
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.
"on at least"
@DevinWalker grammatical changes done. |
No description provided.