Zachary Abresch zacharyabresch

Deleting Branches After Merging

Recently, I completed a pull request and merged my branch into staging once approved. In the past, I've seen some developers delete their branch after merging and others don't do this. I've historically been pretty bad at this myself and was wondering what the team thinks about this.

Personal Opinion

I think that deleting branches after a successful merge should be a practice followed by everyone on the team. There are a lot of good reasons to do [this][1] and there's really no reason not to, IMHO. Currently, on staging, there are 229 branches! (git branch --all | wc -l) Of those 229, 69 of them show as "merged" (git branch --all --merged | wc -l). Running git branch --all is far less useful in this state.

	ready = ->
	# Do stuff
	console.log "READY!!"

	$(document).ready ready
	$(document).on 'page:load', ready

	{
	"version": "4.3.0",

	"navigation" : [
	{
	"name": "Services",
	"slug": "services",
	"sub-nav": [
	{
	"name": "Data Center Services",

	#!/bin/bash

	ln -s ./.gitignore ./.eslintignore && eslint --rule 'array-callback-return: 2' --rule 'complexity: [2, 5]' --rule 'max-statements: [2, 10]' --rule 'max-statements-per-line: [2, {max: 1}]' --rule 'max-nested-callbacks: [2, 2]' --rule 'max-depth: [2, {max: 2}]' --rule 'max-lines: [2, 100]' --rule 'no-return-assign: 2' --rule 'no-param-reassign: 2' --rule 'no-var: 2' --rule 'no-else-return: 2' ./

	#!/bin/bash

	yarn init -y && yarn add --dev babel-core webpack webpack-cli jest-cli babel-jest babel-loader prettier pretty-quick husky babel-preset-env babel-preset-react eslint eslint-config-prettier eslint-config-airbnb eslint-plugin-import eslint-plugin-jsx-a11y eslint-plugin-react && echo '{ "presets": ["env", "react"] }' >> .babelrc && echo 'module.exports = { env: { jest: true }, extends: ["airbnb", "prettier"] };' >> .eslintrc.js && mkdir src && touch src/index.test.js src/index.js && touch README.md && sed -ie -E 's/("license": ".*",)/\1"scripts": { "start": "webpack --mode development", "test": "jest --watchAll" },/g' ./package.json && ./node_modules/.bin/prettier --write ./package.json && subl -an .