Skip to content
This repository has been archived by the owner on Nov 26, 2017. It is now read-only.

Commit

Permalink
Update CONTRIBUTING.markdown
Browse files Browse the repository at this point in the history
  • Loading branch information
dongilbert committed Apr 24, 2013
1 parent 7172fb5 commit 84014ad
Showing 1 changed file with 7 additions and 18 deletions.
25 changes: 7 additions & 18 deletions CONTRIBUTING.markdown
@@ -1,24 +1,13 @@
Contributing to the Joomla! Platform Contributing to the Joomla! Platform
=============== ===============
All contributions are welcome to be submitted for review for inclusion in the Joomla! Platform, but before they will be accepted, we ask that you follow these simple steps:


1) Submitted code must follow the Joomla! Code Standards This repo is no longer accepting contributions or pull requests.
* Please use the resources at http://docs.joomla.org/Coding_style_and_standards for more information about the code standards.


2) Submitted code must not cause existing unit tests to fail The Platform has been merged back in to the (Joomla! CMS)[http://github.com/joomla/joomla-cms],
* If a change in a method causes a change in the expected behavior, the unit test should also be updated to match the change. and the (Joomla! Framework)[http://github.com/joomla/joomla-framework] has taken it's place as
the standalone application framework for Joomla! application.s


3) Unit tests are *highly* encouraged for all pull requests Please determine if your PR should go to the CMS or the Framework, and submit accordingly.
* The Joomla Platform uses PHPUnit for its unit testing. Please review the PHPUnit manual at http://www.phpunit.de/manual/current/en/index.html.
* Whether your pull request is a bug fix or introduces new classes or methods to the Platform, we ask that you include unit tests for your changes
* We understand that not all users submitting pull requests will be proficient with PHPUnit. The maintainers and community as a whole are a helpful group and can help you with writing tests.
* Although bug fixes may be accepted without unit tests (so long as existing tests do not fail with your change), new classes will not be accepted without tests to validate its functionality.


4) Documentation is *highly* encouraged In order to determine where it should go, take a look at the code in the framework src/Joomla folder.
* The Platform Manual is contained in the docs directory of this repo and is written in Markdown format. If you can find an equivilent section of code there, submit it to the Framework. If not, then submit to the CMS.
* When submitting new packages, documentation will be required with your pull request. Please use the existing documentation for examples.
* We understand that not all code is documented at this time. Feel free to expand on the existing documentation by adding to existing chapters or submitting new chapters.

Please be patient as not all items will be tested or reviewed immediately by a Platform maintainer. There is an automated pull request tester online at http://developer.joomla.org/pulls/ which runs the unit tests and checks the code style of every request; please use this resource to ensure your pull does not cause test failures and follows the coding standard.

Lastly, please be receptive to feedback about your change. The maintainers and other community members may make suggestions or ask questions about your change. This is part of the review process, and helps everyone to understand what is happening, why it is happening, and potentially optimize your code.

0 comments on commit 84014ad

Please sign in to comment.