PSA: The large reformat about sofie-core HOT 3 CLOSED

There seems to be some problems developing on Windows now, after the reformat/workflow change:

1. meteor npm run quickformat works only on Unix-based systems (the command syntax used in the script is Unix-specific)
2. the husky may work only on Unix-based systems (the way filenames are piped to the linter may cause an "ENAMETOOLONG" exception, if the number of staged files is significant), there is a suggestion to look up: https://github.com/okonet/lint-staged#using-js-functions-to-customize-tasks which suggest just running the commands on an entire repository if the resulted argument list is expected to be too long (https://github.com/okonet/lint-staged#example-run-eslint-on-entire-repo-if-more-than-10-staged-files).

I'll be looking into fixing these issues, but I just thought I would let anyone else having these issues know. Took me a while to figure out what was wrong.

from sofie-core.

1. Oops, hopefully an easy fix? That is intended to be a temporary command, so feel free to make even more of a mess of it, or perhaps duplicate and make a windows specific version
2. I did digging, and it appears to have been fixed in lint-staged v10 (which requires node 10) lint-staged/lint-staged#732 lint-staged/lint-staged#147. But was also supported for a period up until v9, so the quick fix might be to downgrade lint-staged to v8.2.1 until we get meteor 1.10

from sofie-core.

I think I was able to fix both issues here: #242

from sofie-core.