-
-
Notifications
You must be signed in to change notification settings - Fork 71
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Update docs. * Add support links * Refresh some descriptions and explainations * Grammar & syntax improvements Co-authored-by: John James Jacoby <[email protected]>
- Loading branch information
1 parent
69d3eea
commit a109785
Showing
4 changed files
with
55 additions
and
28 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,29 +1,36 @@ | ||
# Contributing | ||
|
||
Contributions to WP Multi Network are much appreciated. You can help out in several ways: | ||
Contributions are greatly appreciated. You can help in several ways: | ||
|
||
* [File an issue.](https://github.com/stuttter/wp-multi-network/issues/new) | ||
* [Open a pull-request.](https://github.com/stuttter/wp-multi-network/compare) | ||
* [Translate the plugin.](https://translate.wordpress.org/projects/wp-plugins/wp-multi-network) | ||
* [Translate some strings.](https://translate.wordpress.org/projects/wp-plugins/wp-multi-network) | ||
|
||
## Requirements & Recommendations | ||
|
||
When contributing code to WP Multi Network, please keep the folowing in mind: | ||
When contributing code specifically, please keep the following in mind: | ||
|
||
* Write code that is backward-compatible to PHP 5.2 and WordPress 4.6. | ||
* Write code that is backward-compatible to PHP 5.2 and WordPress 4.9. | ||
* Follow the [WordPress coding and documentation standards](https://make.wordpress.org/core/handbook/best-practices/coding-standards/). | ||
* If possible, provide integration tests for your changes. | ||
* If applicable, provide integration tests for your changes. | ||
|
||
WP Multi Network provides easy-to-use workflows for both running integration tests (using PHPUnit) and checking coding and documentation standards (using PHPCodeSniffer). The plugin is integrated with Travis-CI in order to ensure those always pass. | ||
This project provides workflows for: | ||
|
||
* running integration tests (using PHPUnit) | ||
* checking coding and documentation standards (using PHPCodeSniffer). | ||
|
||
It is also integrated with Travis-CI to ensure those always pass. | ||
|
||
### PHPUnit and PHPCS Workflows | ||
|
||
It is recommended to run integration tests and PHPCodeSniffer locally before committing, to check in advance that your changes do not cause unexpected issues. Here is how you can do that: | ||
|
||
* After cloning the plugin, you need to set up its dependencies by running `composer install`, and then `composer config-phpcs` to set up the WordPress Coding Standards. You only need to do this once. | ||
* In order to run the integration tests, you need to run `vendor/bin/phpunit`. | ||
* In order to check against the WordPress Coding Standards, you need to run `vendor/bin/phpcs`. | ||
* `composer install`: Set up all plugin dependencies | ||
* `vendor/bin/phpunit`: Run the integration tests. | ||
* `vendor/bin/phpcs`: Check against the WordPress Coding Standards. | ||
|
||
### Writing Integration Tests | ||
|
||
Integration tests should go into the `tests/integration/tests` directory. Each test class should extend the `WPMN_UnitTestCase` class, and file names should be prefixed with `test-`. | ||
* Integration tests go into the `tests/integration/tests` directory. | ||
* File names must be prefixed with `test-`. | ||
* Each test class must extend the `WPMN_UnitTestCase` class. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
{ | ||
"name": "stuttter/wp-multi-network", | ||
"description": "A Network Management UI for global administrators in WordPress Multisite", | ||
"homepage": "https://github.com/stuttter/wp-multi-network", | ||
"description": "Provides a Network Management Interface for global administrators in WordPress Multisite installations.", | ||
"homepage": "https://wordpress.org/plugins/wp-multi-network/", | ||
"type": "wordpress-plugin", | ||
"license": "GPL-2.0-or-later", | ||
"authors": [ | ||
|
@@ -10,6 +10,11 @@ | |
"email": "[email protected]" | ||
} | ||
], | ||
"support": { | ||
"source": "https://github.com/stuttter/wp-multi-network", | ||
"issues": "https://github.com/stuttter/wp-multi-network/issues", | ||
"forum": "https://github.com/stuttter/wp-multi-network/discussions" | ||
}, | ||
"require": { | ||
"php": ">=5.2", | ||
"composer/installers": "^1.0" | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters