Git Guidelines

From Minetest Developer Wiki
Revision as of 14:02, 9 February 2015 by Celeron55 (talk | contribs)
Jump to navigation Jump to search

This page is mostly directed to core developers with commit access to upstream repositories.

For instructions on basic usage, see Git.

For determining who is allowed to do what, see Organisation.

For guidelines about overall pull request quality, see Merging core pull requests to upstream.

Rules

Upstream branch rules

The master branch is not used for feature freezes. A new feature freeze branch is created when a release is to be made, to which bugfixes are applied until it's considered stable. (It can be extended to be a more complete release model later, but this is the important part for now.) (https://forum.minetest.net/viewtopic.php?f=3&t=11172)

Upstream pull requests and issues

  • If a pull request or an issue does not get a response from its author in one month (when requiring more details), it is closed.
  • If an issue is a duplicate, refer to the first one and close the later ones.

Upstream commit rules

  1. You can push something to upstream [1] only if two members of the core team [2] or the subsystem maintainer agrees on it. (See Organisation.)
  2. Commit messages must start with a capital letter and be in the present tense (look at the commit log)
  3. Do not modify history older than 10 minutes
  4. Use rebase, not merge, to get linear history. [3]

Notes

[1] Upstream is at https://github.com/minetest

[2] The team: https://github.com/minetest?tab=members

[3] For a github pull request, this is easiest done by appending .patch to the pull request URL, wgetting it to your project directory and doing git am whatever.patch. Similarly for single commits.