-
-
Notifications
You must be signed in to change notification settings - Fork 465
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
💅 Regression: Disabling one nursery rule disables all nursery rules #2028
Comments
I don't think this is a bug because as the schema says the If you want to enable all the rules in the nursery group, you should use {
"$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
"formatter": {
"enabled": true
},
"linter": {
"rules": {
"nursery": { "all": true, "noConsole": "off" }
}
}
} However I agree this behavior should be properly documented here: https://biomejs.dev/reference/configuration/#linterrulesall |
@Sec-ant Ah, thanks for the tip. That helps indeed. Nonetheless, this is still a bug, only a different kind of bug then, since |
Yes you're right, I didn't notice that. |
I'm working on this issue 🔨. |
Environment information
Rule name
all nursery rules
Playground link
using CBS, since the playground does not allow modifying the biome config on a rule-level.
https://codesandbox.io/p/devbox/suspicious-hermann-cdnhdx?workspaceId=23165e9f-6758-4f08-8321-45be43925a49
Sidenote: Biome in the CBS is outdated, the installed version there is 1.1.2. Took me a moment to realize that that was why the nursery rules didn't exist.
Expected result
Disabling one nursery rule should not disable all nursery rules. (The following code can also be found in the CBS.)
Code of Conduct
The text was updated successfully, but these errors were encountered: