Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix linebreak-style #69

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
khaninD wants to merge 1 commit into react-bootstrap-table:develop from khaninD:fix
Closed

Conversation

Copy link
Contributor

@khaninD khaninD commented Sep 23, 2017

I use windows and i have CRLF and i have problem with eslint linebreak-style option ... this option cancel of check CRLF or LF linebreak style ...

Copy link
Member

HI @khaninD I think you can try to add .editorconfig and the content can be

end_of_line = lf

Not sure if it is work on widows, but if it's work, please consider to use this solution also it's worthy to commit to code base. thanks

Copy link
Contributor Author

khaninD commented Oct 10, 2017

@AllenFang , eslint show errors.
image

If i added end_of_line = if to .editorconfig,, new files will be with LF style, but old files will be CRLF ...

AllenFang reacted with thumbs up emoji

Copy link
Member

AllenFang commented Oct 11, 2017
edited
Loading

If i added end_of_line = if to .editorconfig,, new files will be with LF style, but old files will be CRLF ...

hmm... that's correct, how about you delete the repo and clone again? or use --fix to fix the legacy CRLF files?

Anyway, I dont think disable the linebreak-style is best solution, it will make our codebase without consistency line break, if above solution is not work for you, I think you can enforce the line breaking on widows with CRLF by:

"linebreak-style": ["error", "windows"]

This solution maybe more safety.

Copy link
Member

Close it due to no response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /