Commit Graph

5 Commits

Author SHA1 Message Date
Uku Taht
1e386f9e10 Update pre-commit versions 2021-11-24 11:16:17 +02:00
Janosh Riebesell
e7593ce94f
Additional pre-commit hooks (#1428)
* add 1st-party langauge agnnostic pre-commit hooks

* white space fixes from pre-commit run -av

* fix CI test/plausible_web/controllers/stats_controller_test.exs#L153

* Revert "fix CI test/plausible_web/controllers/stats_controller_test.exs#L153"

This reverts commit 85c9450efe.

* git checkout 94f476eb~1 -- priv/ref_inspector priv/ua_inspector
2021-11-03 15:46:12 +01:00
Mackenzie
9c2fd9aca5
1. Remove the "airbnb" eslint plugin since it conflicts with prettier (#1374)
and so was just annoying
2. Get rid of all existing ESLint errors.
2a. Turned off `react/display-name` because I couldn't figure out how to
make that wrapped component have a display name. If anyone can figure it
out, that'd be great, because that makes things nicer when using the
React debugger.
2b. The part where it says `plausible()` is undefined in `app.js` is
bothering me. I disabled the check because I can't figure out where that
actually comes from to put in the proper import.
2c. Told ESLint we're using Babel.
3. Added `npm run format` and `npm run lint` commands.
2021-10-11 14:48:19 +02:00
Uku Taht
0de89bad82
Add filter groups (#1167)
* Add filter groups

* New flow for filters

* Visual consistency

* Mobile improvements to dropdown

* Do not let filters wrap on a new line

* Fix country filter

* Add mix format to pre-commit configuration

* Make eslint happy

* Fix formatting

* Use transition from headlessUI
2021-07-21 16:50:26 +03:00
Mackenzie
e95ff80c3d
Add eslint, stylelint, prettier, and pre-commit (#484)
- When you try to commit, Pre-commit will run checks for all three, if
  installed, _only_ on staged files, and prevent committing if the linters are
  unhappy.
- Several eslint rules that generate a lot of errors are disabled for now.
  That's to allow a more gradual transition, so you won't change 1 line and be
  told to fix 100 other lines (but maybe like 20 because there are still a lot
  of rules enabled).
- Prettier is set to require pragma. That's also to allow a more gradual
  transition. As each file is tidied up, run Prettier on it, and it'll add a
  special comment that tells it that it's now responsible for keeping that one
  tidy.
2020-12-23 10:45:53 +02:00