Skip to content
New issue

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

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

Already on GitHub? # to your account

disable multiline-comment-style rule #43

Merged
merged 1 commit into from
Dec 14, 2023
Merged

Conversation

davidchambers
Copy link
Member

There are valid reasons to use /* ... */ comments even in codebases that generally use // comments.

We currently disable multiline-comment-style in many places:

$ fgrep -l '"multiline-comment-style": ["off"]' **/.eslintrc.json
sanctuary-def/.eslintrc.json
sanctuary-descending/.eslintrc.json
sanctuary-either/.eslintrc.json
sanctuary-identity/.eslintrc.json
sanctuary-maybe/.eslintrc.json
sanctuary-pair/.eslintrc.json
sanctuary-type-classes/.eslintrc.json
sanctuary-type-identifiers/.eslintrc.json
sanctuary/.eslintrc.json

We want a style guide rather than style rules.

@davidchambers davidchambers merged commit 4416b6f into main Dec 14, 2023
1 check passed
@davidchambers davidchambers deleted the davidchambers/eslint branch December 14, 2023 07:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant