Skip to content

Conversation

Bashamega
Copy link
Contributor

No description provided.

@saschanaz
Copy link
Contributor

I'm happy with this.

@jakebailey
Copy link
Member

I thought these rules had been removed from eslint proper? I guess this repo is very behind on linting versions?

@saschanaz
Copy link
Contributor

Hmm? It's a thing? https://eslint.org/docs/latest/rules/curly

@HolgerJeromin
Copy link
Contributor

HolgerJeromin commented Oct 5, 2025

I thought these rules had been removed from eslint

curly is not part of the stylistics rules which were migrated to another team.

https://eslint.style/rules

@jakebailey
Copy link
Member

Interesting choice to only get rid of some style rules...

@jakebailey
Copy link
Member

Personally I think doing prettier or dprint would be better solutions, but, it's all the same

@jakebailey jakebailey merged commit 9b934cb into microsoft:main Oct 5, 2025
7 checks passed
@HolgerJeromin
Copy link
Contributor

@saschanaz
Copy link
Contributor

We do use prettier already, except it didn't say anything here.

@Bashamega Bashamega deleted the eslint branch October 6, 2025 02:53
@HolgerJeromin
Copy link
Contributor

We could add the plugin:
https://github.com/JoshuaKGoldberg/prettier-plugin-curly

@saschanaz
Copy link
Contributor

If we used prettier only, yeah. Not much point if we have eslint IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants