saiichihashimoto / lint-my-app

So you can write your app instead of lint it!

So you can write your app instead of lint it!

lint-staged keeps the 💩 out of your app. I got tired of configuring husky and lint-staged to get started. lint-my-app sets sane linting defaults so you can write your app instead of lint it!

Install

npm install --save-dev lint-my-app husky

// .huskyrc.js
module.exports = require('lint-my-app/husky');

git commit -m 'Keep calm and lint'

Features

• Fixes files using lint-staged on commit.
• Lints your entire codebase on push.
• All batteries included (except husky configuration).
• Uses your personal eslint and stylelint configs
• Respects .gitignore

Commands

For the most part, lint-my-app/husky can be given to husky and you're done! But the internal commands are open to you!

lint-my-app lint

This lints your entire app.

lint-my-app fix

This fixes your entire app.

lint-my-app staged

This fixes your git staged files.

lint-my-app's Issues

Print warnings

Error Codes

I don't think I'm doing them right

Tests

eslint/stylelint only when files exist

The globbing from the tools are kind of shaky when it comes to dotfiles and it's weird to see the tasks run even when there's no files.

• Don't forget their .*ignore files.

Include ALL package.json's

• in staged
• in fix
• in lint

Sketch files into images

Include all package.json's for pkg-ok

• In staged
• In lint

validate codecov yaml

https://docs.codecov.io/docs/codecov-yaml#section-validate-your-repository-yaml

Reuse all of these commonalities with fix

Probably test them too

---

lint-my-app/src/lint-staged.config.js

Lines 12 to 17 in 6f03a88

	* TODO Reuse all of these commonalities with fix
	* BODY Probably test them too
	*/
	// Can't use "export default ..." with lint-staged
	module.exports = {

---

This issue was generated by todo based on a TODO comment in 6f03a88 when #205 was merged. cc @saiichihashimoto.

End to End test/Integration test

I'm relying on too many other packages (ie lint-staged) for their changes to keep breaking me.

• Success case exit 0
• Failure case exit 1

Markdown files

Validate Travis yml

Use minimal configs by default

• Function without an eslint or stylelint config
• Use a least restrictive config for each, purely for syntax correctness

Node instead of bash

Run on dotfiles properly

eslint doesn't actually run on ".*" files currently without the pattern fixing. Need to do that, and check that all other runners also handle that properly