Talk to People
Always check for existing issues or pull-requests first. Comment or vote for them.
Please create an issue first and let us discuss this change, before you do any changes. Not all changes will be accepted by the maintainer of the project, you can save a lot of time if you talk to us, before start coding.
Gitter.im is also a good place to throw arround ideas in an informal way. Lets talk to each other, to improve PhileCMS.
Patch / Pull Request Workflow
- Fork Phile on github
- Make your changes
- Run the test cases locally (call
./phingin the root and choose "run test suite"), make sure no test fails
- Create a pull request on github into the
Travis CIwill build and test your pull request automatically, if all tests passed go to next step
- Each pull request has to be reviewed by two other people. If the pull request is reviewed by at least two people (give +1 in the comments), one of the maintainer will accept the request and merge it into the core.
The maintainers also has a veto, in cases where the code structure is not Ok or an other pull request will add the same feature.
For each pull request please use one of the following prefix:
If you implement a new feature, please use this prefix in the title of the pull request.
If you fix any bug, please use this prefix in the title of the pull request.
If you fix a already documented bug, pleaser refer to the issue in the issuetracker.
GIT Commit Message Template
Please use the following GIT commit message template, for any pull request.
You can use a custom template for getting the commit message faster done.
All you need is the template below and the command
git config --local commit.template .gitmessage.txt
We have added the
.gitmessage.txt file to the root of the project.
[BUGFIX|FEATURE|TASK] Short summary of changes introduced by this patch More detailed explanatory text, if necessary. Resolves: #12345 (refer to github issue number) Documentation: #12346 (refer to github issue number, where a documentation is available) Releases: 0.9.3 (this should be the next version number, please ask the core developer before you fill in this information or leave it blank Related: #1234, #1235 list here all related github issue numbers