-
Notifications
You must be signed in to change notification settings - Fork 226
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #35 from thellimist/patch-2
Add Contribution Document
- Loading branch information
Showing
2 changed files
with
21 additions
and
1 deletion.
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 |
---|---|---|
@@ -0,0 +1,17 @@ | ||
# Contributing to Cloudflare PHP API Binding | ||
|
||
ππ First off, thanks for taking the time to contribute! ππ | ||
|
||
## How To Contribute | ||
|
||
We welcome community contribution to this repository. To help add functionality or address issues, please take the following steps: | ||
|
||
* Fork the repository from the master branch. | ||
* Create a new branch for your features / fixes. | ||
* Make the changes you wish to see. | ||
* Add tests for all changes. | ||
* Run `make fix` command to ensure the code is styled. | ||
* Run `make test` command to ensure the tests pass. | ||
* Create a pull request with details of what changes have been made and explanation of new behaviour. | ||
* Ensure documentation contains the correct information. | ||
* Pull requests will be reviewed and hopefully merged into a release. |
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