search

webgpu / webgpu-debugger

MIT License
69 stars 8 forks source link

Setup eslint and auto-formatting #14

Closed greggman closed 2 years ago

greggman commented 2 years ago

Note: This is lightweight auto-formatting. If you want full autoformatting we can add "prettier" but it's super opinionated (very little configuration except tabs, spaces, line length, single/double quotes. That's like it.

This version formats on save, the entire file. But it mostly does indenting and I suppose eslint features that have a "quick-fix" option.

I didn't actually reformat any files. I thought I'd just post this if you'd prefer it.

Also you can change any rules in .eslintrc.cjs. I just grabbed one from another project of mine and changed it to 4 spaces.

greggman commented 2 years ago

oh, it assumes you're using VSCode since it's format on save. If I put in prettier then there'd also be a command to format.

Kangz commented 2 years ago

We use prettier in the CTS and it works well I think. I personally use VIM so I don't really have formatting on save, but a command line used pre-commit would work well for me.

greggman commented 2 years ago

oh, I'll add prettier to this PR

Kangz commented 2 years ago

I mean, as you wish, it's just that I'd personally be using a command line tool only :/