Inline eslint configuration?

[jordrake]

I noticed there are some inconsistencies across the packages (e.g. single and double quotes) and saw that this uses an external but specific eslint config located at https://github.com/ben-eb/eslint-config-cssnano.

I am assuming at one point this was because the packages used to be spread out across various repositories. Now this is no longer the case (I believe?) could we bring the eslint configuration inline either as a top-level .eslintrc or a packages/eslint-config-cssnano to make it easier to update.

Or just use a shared one.

[alexander-akait]

@jordrake Yes, we need do it, also we should use prettier https://github.com/prettier/prettier for code style. I think we can avoid creating new package because we in monorepo. Feel free to send a PR.

Also will be great to switch from ava to jest with snapshot testing, he is faster and snapshot testing allow writing test very fast and very conveniently. Ava also have snahpshots, but the last time when i use their they were very unstable.

[jordrake]

You'll get no argument from me on any of those fronts.

The conversion from jest to ava might be slightly painful as the tests rely a lot of ava's macro functionality but we could add a utility function which has the same signature or do something clever with codemods.

[alexander-akait]

@jordrake yep, so migration from ava to jest is not high priority. We have other high priority bugs #411 #458 #535 (require more work). Any help welcome.

[andyjansson]

#411 #458

These two I'll fix. Declaration sorting I haven't started on.