Changes between Version 24 and Version 25 of DevWikiComitGuidelines

Timestamp:

Feb 24, 2011, 5:12:03 AM (13 years ago)

Author:

robe

Comment:

—

DevWikiComitGuidelines

@@ -1 +1 @@
 = RFC ? : PostGIS Committer Guidelines =
-'''Version''': 1.00000002
-
+'''Version''': 1.00000003
 '''Status''': draft
 
@@ -43 +42 @@
     * Use meaningful descriptions for commit log entries.
     * Add a bug reference like "(#1232)" at the end of SVN commit log entries when committing changes related to a ticket in Trac. The '#' character enables Trac to create a hyperlink from the changeset to the mentionned ticket.
-    * Add a note HISTORY to the commit log if this is a major feature that should be noted in the NEWS version release and credits
+    * When new enhancements are added or breaking changes are made, these should be noted in the trunk/NEWS or relevant branch/NEWS as soon as conveniently possible.  The note should include the trac # and contributors to the feature/change
     * After commiting changes related to a ticket in Trac, write the tree and revision in which it was fixed in the ticket description. Such as "Fixed in trunk (r12345) and in branches/1.7 (r12346)". The 'r' character enables Trac to create a hyperlink from the ticket to the changeset.
     * Changes should not be committed in stable branches without a corresponding bug id. Any change worth pushing into the stable version is worth a bug entry.
@@ -51 +50 @@
     * Significant changes to the main development version should be discussed on the postgis-dev list before you make them, and larger changes will require an RFC approved by the PSC.
     * Do not create new branches (except for spike branches) without the approval of the PSC. Release managers are assumed to have permission to create a branch.
-    * spike branche (htose in the spike/username area are to be used for experimentation of for major code refactorings that will destabilize the trunk.  After
-such experimentation is stable, this can then be merged into the trunk after approval from PSC members.
+    * spike branch (those in the spike/username area are to be used for experimentation or for major code refactorings that will destabilize the trunk.  After such experimentation is stable, this can then be merged into the trunk after approval from PSC members.
     * All source code in SVN should be in Unix text format as opposed to DOS text mode.
     * C code should follow our designated A-style format and astyle should be applied before commit
     * When committing new features or significant changes to existing source code, the committer should take reasonable measures to insure that the source code continues to build and work on the most commonly supported platforms (currently Linux and Windows), either by testing on those platforms directly, running Buildbot tests, or by getting help from other developers working on those platforms. If new files or library dependencies are added, then the configure.in, Makefile.in, Makefile.vc and related documentations should be kept up to date.
-    * In the event of buildbot failure notice, the person who caused the break must fix the break before working on anything else 
+    * In the event of broken build (build bot fail notification), the person who broke the build must fix the break before working on anything else 
 
 == Relationship with other Projects ==