Movatterモバイル変換


[0]ホーム

URL:


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

Configs: Last round of "final" changes to configs for v6#7130

Discussion options

Following#6014, I described the changes in#7110 as the"final v6 changes to configs". That was inaccurate - it was the final changes to configsbefore trying them out in real-world projects (#6760). Now that we've tried things out, there are a few more changes I think we should make...

Increasing strictness:

  • no-explicit-any: It was moved fromrecommended tostrict, but I think we should move it back. Most projects shouldn't be usingany.
  • strict-boolean-expressions: SwitchallowNullableEnum tofalse by default, perChanges to configurations for 6.0.0 #6014 (comment)
    • (this was originally listed under the following section erroneously - thanks@tylerlaprade for pointing that out!)

Decreasing strictness:

  • no-confusing-void-expressions: It was added tostylistic-type-checked but I think it's too opinionated for many users, especially for those working in JSX-based frameworks. I think we should remove it fromstylistic-type-checked for now.
  • no-mixed-enums: It was added torecommended, but I think it's a rare enough edge case that a large percentage of consumers wouldn't care for the rule. I think we should instead move it tostrict for now.(this is instrict-type-checked, I was wrong)
  • sort-type-constituents: It was added tostylistic. It's pretty nitpicky without actual bug-catching and I think many users would be by it. I think we should remove it fromstylistic for now.

...anything else that's come up & we should adjust for?

You must be logged in to vote

Replies: 2 comments 1 reply

Comment options

JoshuaKGoldberg
Jun 26, 2023
Maintainer Author

Coming over from#7138:

You must be logged in to vote
0 replies
Comment options

Technically thestrict-boolean-expressions: allowNullableEnum change should be listed under the "Increasing strictness" section. Thanks for including it, by the way!

You must be logged in to vote
1 reply
@JoshuaKGoldberg
Comment options

JoshuaKGoldbergJul 12, 2023
Maintainer Author

Whoops, thanks! Edited :)

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Labels
None yet
2 participants
@JoshuaKGoldberg@tylerlaprade

[8]ページ先頭

©2009-2025 Movatter.jp