search

webcompat / webcompat.com

Source code for webcompat.com
https://webcompat.com
356 stars 189 forks source link

editorconfig audit #2201

Open miketaylr opened 6 years ago

miketaylr commented 6 years ago

There's a lot of code that doesn't obey our editorconfig settings, and we don't fail tests due to this, but I guess it's nice to be consistent.

For these types of cleanups, I suppose doing a little at a time is the right approach.

There's a "checker" here that I just used that seems to work well.

https://github.com/editorconfig-checker/editorconfig-checker.javascript

Maybe not worth worrying about python (since pep8 will do that) or HTML.

Maybe once we get things cleaned up, we can add this to our linter script, and have it fail tests.

zoepage commented 6 years ago

It'd be great to also keep JSON(for e.g. fixtures) in mind as we have the same issues there.