universityofnottingham / dotnet-boilerplate Goto Github PK

What it says

In the js gulp task, browserify should be run with the option debug: true so the source maps generated are actually useful.

This isn't a concern for production since we don't put source maps in our minified files.

Husky will be deprecating the use of npm's pre scripts for defining hooks.

Instead husky hooks should be defined in a separate config file or object inside package.json

we should move our precommit hook to the new config format in package.json as follows:

{
  //...
  "husky": {
    "hooks": {
      "pre-commit": "..."
    }
  }
  //...
}

Remove that line - FA5's SVGs are all JavaScript all the time

Gulp 4 is out and i've successfully used it on a js lib, so we could update this repo to use it

While no custom sass in the boilerplate requires vendor prefixes, we do import bootstrap 4's sass from source, which is expected to be run through Autoprefixer (as per the dist build process in the bootstrap 4 repo).

So we need to run it through Autoprefixer in the gulp task we build our sass in.

• use prettier for formatting
• precommit hook
• remove formatting rules from the linters to focus them on code quality
• makes the linting experience less painful

when it's out