@springtree/coding

SpringTree coding guidelines

This repository contains information about our company rules and guidelines when it comes to writing software. Our primary development language is JavaScript/TypeScript so those will be featured here prominently. A collection of configuration files for various tools and linters that can be used in our projects can also be found here.

You can now use npx to setup linting and the gitcommit hooks using this repository:

npx @springtree/coding --init

If you are using a project without TypeScript run:

npx @springtree/coding --init --skip-ts

Style guide

We have adopted the Airbnb style guide for our JavaScript and TypeScript projects. This is a very widely used standard and support is available in a multitude of editors and CI tools. We have added the following rules to the style to make the resulting code a little more spacious and readable:

• max line length increased to 160
• enforce spaces inside a parenthesis
• allow console statements

All new projects must use this style guide. When coding on an existing project stick to the current style used throughout that project if there are too many issues when switching. This repository contains configuration files for eslint and tslint.

You can install these tools using npm:

npm i -D eslint eslint-config-airbnb eslint-plugin-jsx-a11y eslint-plugin-react eslint-plugin-import tslint tslint-config-airbnb

Human language

All code, variable names, code comments and documentation should be written in English.

Project README

Every project needs to have a README.md (or equivalent) that must contain:

• the name and purpose of the project
• how to run the project
• how to build the project

Any additional information about how to run unit tests should be added if available.

Git flow AVH

We use Git flow AVH as our branching strategy. You can install this on a Mac using Homebrew

brew install git-flow-avh

We have a whole wiki page dedicated to how we use Git flow to perform releases. Try to contain work in separate feature branches as much as possible.

Git branch protection and enforcement

Projects on GitHub should be set up with branch protection to now allow direct pushes to the master and develop branches. The restriction on master is also enforced for administrators. We have our own git branch name checking tool.

Branch naming checking can be installed with:

npx @springtree/coding --gitflow

NOTE: Be sure to initialize your repo with git-flow first and perform the initial push of master and develop branches.

Git commit log format

We use the Angular commit log format which we enforce using a combination of commitlint and husky.

You can install these tools using npm:

npm i -D husky @commitlint/cli @commitlint/config-angular

Add the following husky hook to your package.json to enforce the format:

  "husky": {
    "hooks": {
      "commit-msg": "commitlint -E HUSKY_GIT_PARAMS"
    }
  },

Create a commitlint configuration files:

echo "module.exports = { extends: ['@commitlint/config-angular'] }" > commitlint.config.js

Build using CI

All projects should be built using a CI and should not depend on the build chain of an individual developers laptop. This should preferably be set up at project inception.

We use the folloing CI's at this time:

• Bitrise.io (for mobile)
• TravisCI (general purpose)
• CloudBuild (Google projects)

Use tools like docker to encapsulate your build chain.

Pull request validation with CI

Pull requests should be set up to use the CI to validate the branch is building. If available this should include running the unit test suite.