Skip to content

Latest commit

 

History

History
63 lines (43 loc) · 2.57 KB

workflow.md

File metadata and controls

63 lines (43 loc) · 2.57 KB

Git and GitHub Workflow

The following diagram visualizes our branching model:

Branching Model

Our git workflow looks as follows:

  • The main branch reflects the latest state of development, and should always compile.

  • In case we need to release a hotfix, we use dedicated patch release branches.

  • The stable branch always points to the latest release that is not a release candidate. It exists so support a streamlined workflow for some packaging tools (e.g., Nix).

  • For new features or fixes, use topic branches that branch off main with a naming convention of topic/description. After completing work in a topic branch, check the following steps to prepare for a merge back into main:

    • Squash your commits such that each commit reflects a self-contained change. You can interactively rebase all commits in your current pull request with git rebase --interactive $(git merge-base origin/main HEAD).

    • Create a pull request to main on GitHub.

    • Wait for the results of continuous integration tools and fix any reported issues.

    • Ask a maintainer to review your work when your changes merge cleanly. If you don't want a specific maintainer's feedback, ask for a team review from tenzir/engineering.

    • Address the feedback articulated during the review.

    • A maintainer will merge the topic branch into main after it passes the code review.

  • Similarly, for features or fixes relating to a specific GitHub issue, use topic branches that branch off main with a naming convention of topic/XXX, replacing XXX with a short description of the issue.

Commit Messages

Commit messages are formatted according to this git style guide.

  • The first line succinctly summarizes the changes in no more than 50 characters. It is capitalized and written in and imperative present tense: e.g., "Fix a bug" as opposed to "Fixes a bug" or "Fixed a bug". As a mnemonic, prepend "When applied, this commit will" to the commit summary and check if it builds a full sentence.

  • The first line does not contain a dot at the end. (Think of it as the header of the following description).

  • The second line is empty.

  • Optional long descriptions as full sentences begin on the third line, indented at 72 characters per line, explaining why the change is needed, how it addresses the underlying issue, and what side-effects it might have.