Docutils | Overview | About | Users | Reference | Developers

Docutils Project Policies

Author:

David Goodger; open to all Docutils developers

Contact:
docutils-develop@lists.sourceforge.net
Date:
$Date$
Revision:
$Revision$

The Docutils project group is a meritocracy based on code contribution and lots of discussion [1]. A few quotes sum up the policies of the Docutils project. The IETF's classic credo (by MIT professor Dave Clark) is an ideal we can aspire to:

We reject: kings, presidents, and voting. We believe in: rough consensus and running code.

As architect, chief cook and bottle-washer, David Goodger currently functions as BDFN (Benevolent Dictator For Now). (But he would happily abdicate the throne given a suitable candidate. Any takers?)

Eric S. Raymond, anthropologist of the hacker subculture, writes in his essay The Magic Cauldron:

The number of contributors [to] projects is strongly and inversely correlated with the number of hoops each project makes a user go through to contribute.

We will endeavour to keep the barrier to entry as low as possible. The policies below should not be thought of as barriers, but merely as a codification of experience to date. These are "best practices"; guidelines, not absolutes. Exceptions are expected, tolerated, and used as a source of improvement. Feedback and criticism is welcome.

As for control issues, Emmett Plant (CEO of the Xiph.org Foundation, originators of Ogg Vorbis) put it well when he said:

Open source dictates that you lose a certain amount of control over your codebase, and that's okay with us.

Python Coding Conventions

Contributed code will not be refused merely because it does not strictly adhere to these conditions; as long as it's internally consistent, clean, and correct, it probably will be accepted. But don't be surprised if the "offending" code gets fiddled over time to conform to these conventions.

The Docutils project shall follow the generic coding conventions as specified in the Style Guide for Python Code (PEP 8) and Docstring Conventions (PEP 257), summarized, clarified, and extended as follows:

Documentation Conventions

Copyrights and Licensing

The majority of the Docutils project code and documentation has been placed in the public domain (see Copying Docutils).

Unless clearly and explicitly indicated otherwise, any patches (modifications to existing files) submitted to the project for inclusion (via Subversion, SourceForge trackers, mailing lists, or private email) are assumed to be in the public domain as well.

Any new files contributed to the project should clearly state their intentions regarding copyright, in one of the following ways:

Repository

Please see the repository documentation for details on how to access Docutils' Subversion repository. Anyone can access the repository anonymously. Only project developers can make changes. (If you would like to become a project developer, just ask!) Also see Setting Up For Docutils Development below for some useful info.

Unless you really really know what you're doing, please do not use svn import. It's quite easy to mess up the repository with an import.

Branches

(These branch policies go into effect with Docutils 0.4.)

The "docutils" directory of the trunk (a.k.a. the Docutils core) is used for active -- but stable, fully tested, and reviewed -- development.

If we need to cut a bugfix release, we'll create a maintenance branch based on the latest feature release. For example, when Docutils 0.5 is released, this would be branches/docutils-0.5, and any existing 0.4.x maintenance branches may be retired. Maintenance branches will receive bug fixes only; no new features will be allowed here.

Obvious and uncontroversial bug fixes with tests can be checked in directly to the core and to the maintenance branches. Don't forget to add test cases! Many (but not all) bug fixes will be applicable both to the core and to the maintenance branches; these should be applied to both. No patches or dedicated branches are required for bug fixes, but they may be used. It is up to the discretion of project developers to decide which mechanism to use for each case.

Feature additions and API changes will be done in feature branches. Feature branches will not be managed in any way. Frequent small check-ins are encouraged here. Feature branches must be discussed on the docutils-develop mailing list and reviewed before being merged into the core.

Review Criteria

Before a new feature, an API change, or a complex, disruptive, or controversial bug fix can be checked in to the core or into a maintenance branch, it must undergo review. These are the criteria:

  • The branch must be complete, and include full documentation and tests.

  • There should ideally be one branch merge commit per feature or change. In other words, each branch merge should represent a coherent change set.

  • The code must be stable and uncontroversial. Moving targets and features under debate are not ready to be merged.

  • The code must work. The test suite must complete with no failures. See Docutils Testing.

The review process will ensure that at least one other set of eyeballs & brains sees the code before it enters the core. In addition to the above, the general Check-ins policy (below) also applies.

Check-ins

Changes or additions to the Docutils core and maintenance branches carry a commitment to the Docutils user community. Developers must be prepared to fix and maintain any code they have committed.

The Docutils core (trunk/docutils directory) and maintenance branches should always be kept in a stable state (usable and as problem-free as possible). All changes to the Docutils core or maintenance branches must be in good shape, usable, documented, tested, and reasonably complete. Starting with version 1.0, they must also comply with the backwards compatibility policy.

  • Good shape means that the code is clean, readable, and free of junk code (unused legacy code; by analogy to "junk DNA").

  • Usable means that the code does what it claims to do. An "XYZ Writer" should produce reasonable XYZ output.

  • Documented: The more complete the documentation the better. Modules & files must be at least minimally documented internally. Docutils Front-End Tools should have a new section for any front-end tool that is added. Docutils Configuration Files should be modified with any settings/options defined. For any non-trivial change, the HISTORY.rst file should be updated.

  • Tested means that unit and/or functional tests, that catch all bugs fixed and/or cover all new functionality, have been added to the test suite. These tests must be checked by running the test suite under all supported Python versions, and the entire test suite must pass. See Docutils Testing.

  • Reasonably complete means that the code must handle all input. Here "handle" means that no input can cause the code to fail (cause an exception, or silently and incorrectly produce nothing). "Reasonably complete" does not mean "finished" (no work left to be done). For example, a writer must handle every standard element from the Docutils document model; for unimplemented elements, it must at the very least warn that "Output for element X is not yet implemented in writer Y".

If you really want to check code directly into the Docutils core, you can, but you must ensure that it fulfills the above criteria first. People will start to use it and they will expect it to work! If there are any issues with your code, or if you only have time for gradual development, you should put it on a branch or in the sandbox first. It's easy to move code over to the Docutils core once it's complete.

It is the responsibility and obligation of all developers to keep the Docutils core and maintenance branches stable. If a commit is made to the core or maintenance branch which breaks any test, the solution is simply to revert the change. This is not vindictive; it's practical. We revert first, and discuss later.

Docutils will pursue an open and trusting policy for as long as possible, and deal with any aberrations if (and hopefully not when) they happen. We'd rather see a torrent of loose contributions than just a trickle of perfect-as-they-stand changes. The occasional mistake is easy to fix. That's what version control is for!

Version Identification

The state of development of the current Docutils codebase is stored in two forms: the sequence docutils.__version_info__ and the PEP 440 conformant text string docutils.__version__. See also the Docutils Release Procedure

docutils.__version_info__

docutils.__version_info__ is an instance of docutils.VersionInfo based on collections.namedtuple. It is modelled on sys.version_info and has the following attributes:

majornon-negative integer

Major releases (x.0, e.g. 1.0) will be rare, and will represent major changes in API, functionality, or commitment. The major number will be bumped to 1 when the project is feature-complete, and may be incremented later if there is a major change in the design or API. When Docutils reaches version 1.0, the major APIs will be considered frozen. For details, see the backwards compatibility policy.

minornon-negative integer

Releases that change the minor number (x.y, e.g. 0.5) will be feature releases; new features from the Docutils core will be included.

micronon-negative integer

Releases that change the micro number (x.y.z, e.g. 0.4.1) will be bug-fix releases. No new features will be introduced in these releases; only bug fixes will be included.

The micro number is omitted from docutils.__version__ when it equals zero.

releaseleveltext string

The release level indicates the development status (or phase) of the project's codebase:

Release Level

Label [2]

Description

alpha

a

Reserved for use after major experimental changes, to indicate an unstable codebase.

beta

b

Indicates active development, between releases.

candidate

rc

Release candidate: indicates that the codebase is ready to release unless significant bugs emerge.

final

Indicates an official project release.

serialnon-negative integer

The serial number is zero for final releases and incremented whenever a new pre-release is begun.

releaseboolean

True for official releases and pre-releases, False during development.

  • One of {major, minor, micro, serial} is incremented after each release, and the lower-order numbers are reset to 0.

  • The default state of the repository during active development is releaseĀ level = "beta", serial = 0, release = False.

docutils.__version_info__ can be used to test for a minimally required version, e.g.

docutils.__version_info__ >= (0, 13)

is True for all versions after "0.13".

docutils.__version__

The text string docutils.__version__ is a human readable, PEPĀ 440-conforming version specifier. For version comparison operations, use docutils.__version_info__.

docutils.__version__ takes the following form:

"<major>.<minor>[.<micro>][<releaselevel>[<serial>]][.dev]"
 <--- release segment ---><-- pre-release segment -><- development ->
  • The pre-release segment contains a label representing the releaselevel ("a", "b", or "rc") and eventually a serial number (omitted, if zero).

  • The development segment is ".dev" during active development (release == False) and omitted for official releases and pre-releases.

Examples of docutils.__version__ identifiers, over the course of normal development (without branches), in ascending order:

Release Level

Version Identifier

final (release)

0.14

beta (development) [3]

0.15b.dev

beta (release) [4]

0.15b

candidate 1 (dev.)

0.15rc1.dev

candidate 1 (release)

0.15rc1

candidate 2 (dev.) [4]

0.15rc2.dev

candidate 2 (release) [4]

0.15rc2

...

final (release)

0.15

beta (development) [3]

0.16b.dev

Policy History

  • Prior to version 0.4, Docutils didn't have an official version numbering policy, and micro releases contained both bug fixes and new features.

  • An earlier version of this policy was adopted in October 2005, and took effect with Docutils version 0.4.

  • This policy was updated in June 2017 for Docutils version 0.14. See Feature Request #50 and the discussion on docutils-devel from May 28 to June 20 2017.

Backwards Compatibility Policy

Docutils' backwards compatibility policy follows the rules for Python in PEP 387.

A majority of projects depends on Docutils indirectly, via the Sphinx document processor.

Changes that may affect end-users (e.g. by requiring changes to the configuration file or potentially breaking custom style sheets) should be announced with a FutureWarning.

Snapshots

Snapshot tarballs can be downloaded from the repository (see the "download snapshot" button in the head of the code listing table).

Setting Up For Docutils Development

When making changes to the code, testing is a must. The code should be run to verify that it produces the expected results, and the entire test suite should be run too. The modified Docutils code has to be accessible to Python for the tests to have any meaning. See editable installs for ways to keep the Docutils code accessible during development.

Mailing Lists

Developers are recommended to subscribe to all Docutils mailing lists.