Pull requests

#88 Merged
Repository
jonwaltman/sphinx-devguide sphinx-devguide
Branch
default
Repository
birkenfeld/sphinx sphinx
Branch
default

Add a Sphinx developer's guide

Author
  1. Jon Waltman
Reviewers
Description

This adds a new document to describe the development process of Sphinx.

Comments welcome.

  • Learn about pull requests

Comments (3)

  1. Jon Waltman author

    I'm not sure how draconian the policy should be about code review and the accepting of pull requests.

    How about anyone with commit access can merge any pull request after approving it (including their own if they deem appropriate but normally not).

    I don't think we need to require creating pull requests for trivial changes. What constitutes "trivial" is up to the committer.