hoverzoom icon indicating copy to clipboard operation
hoverzoom copied to clipboard

code cleanup

Open Mattwmaster58 opened this issue 4 years ago • 1 comments

The codebase is full of small formatting annoyances, I think something like prettier could enhance the readability. I'd be happy to put in a PR if the maintainers are on board.

Examples:

<function name> : function () {}
// instead of 
function <function name> () {}
<val> == <other val>
//instead of
<val> === <other val>

and many others

Admittedly, most of the annoyance is born out of my IDE complaining about things, and they are tolerable, so this isn't the most important thing to address. And some of these are stylistic choices (eg, the object example above)

Mattwmaster58 avatar Jun 12 '21 16:06 Mattwmaster58

I don't mind, maybe something like https://github.com/marketplace/actions/prettier-action can be used, although I have never personally tried github actions so I don't know of there are any gotchas.

extesy avatar Jun 12 '21 21:06 extesy