You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have a team experimenting with this repo and we are using a mix of Windows and Macs for development. If the Windows users touch any files, they get a linebreak-style error during the build portion of an npm run serve, for example:
There are straight-forward work-arounds but I think this issue should be addressed in your default branch as I assume this could be a blocker for newer users. Here are a few options to consider:
Disable the lint rule by adding 'linebreak-style': 'off', to the repo's .eslintrc.js
Add instructions to your README for how to configure popular editors to set the end of line character you expect. For instance, for VSCode configure setting: "files.eol": "\n".
If you'd like me to submit a PR using any of these suggestions, I'm happy to do so.
The text was updated successfully, but these errors were encountered:
@johnpduane I'm leaning towards not using linter as a code formatter at all and remove all formatting rules from the linter (airbnb). Add prettier rules for code formatting but don't enforce them. The team working on the project can set format on save and external contributors code can be formatted on merge.
I have a team experimenting with this repo and we are using a mix of Windows and Macs for development. If the Windows users touch any files, they get a linebreak-style error during the build portion of an
npm run serve
, for example:There are straight-forward work-arounds but I think this issue should be addressed in your default branch as I assume this could be a blocker for newer users. Here are a few options to consider:
'linebreak-style': 'off',
to the repo's .eslintrc.jsend_of_line
, see airbnb/javascript/.editorconfig for an example."files.eol": "\n"
.If you'd like me to submit a PR using any of these suggestions, I'm happy to do so.
The text was updated successfully, but these errors were encountered: