Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

This document will go over GitHub branching best practices.

Basic Flow

The two primary branches on the tech dashboard are any repo we maintain should be dev and master. Neither branch will be directly committed to and will be a protected branch. Only PRs will be used to alter dev and master. To make a new feature, a branch will be created based off of dev and then a PR will be made into dev once it is completed. Later dev and master will be synced via a PR. PRs into dev will require one reviewer and PRs into master will require 2 reviewers.

Naming Conventions

  • Feature branches should be named “feature/<a_descriptive_name>”.

  • Hotfix branches should be named “hotfix/<a_descriptive_name>”

...

  • .

Pull Request Best Practices

  • All pull requests should be reviewed by another member of the team who has a basic understanding of the project.

  • Each PR into dev should contain a single github issue and link that issue

...

  • . Each PR into master should link all issues that are contained within the PR.

  • The description should contain a bulleted list of changes to make the PR review faster and easier

...

Deleting Branches After Merge

...

  • .

  • Once a feature or hotfix branch has been merged it should be deleted.

Helpful Links