generated from JoshuaKGoldberg/create-typescript-app
-
Notifications
You must be signed in to change notification settings - Fork 19
Closed
Labels
package: coreRelated to the core Flint package underlying all other packages and plugins.Related to the core Flint package underlying all other packages and plugins.status: needs investigationFurther research required...? 🔎Further research required...? 🔎type: bugSomething isn't working 🐛Something isn't working 🐛
Description
Bug Report Checklist
- I am using the latest published version of all Flint packages.
- I have searched for related issues and found none that matched my issue.
Expected
A config like this should disable the nonNullAssertions TS rule:
import { defineConfig, ts } from "flint";
export default defineConfig({
use: [
{
files: ts.files.all,
rules: [
ts.presets.logicalStrict,
ts.presets.stylisticStrict,
ts.rules({
// We have the typescript-eslint rule already;
// disabling two rules on a line is a lot
nonNullAssertions: false,
}),
],
},
],
});Actual
As seen in https://github.com/flint-fyi/flint/actions/runs/21182525010/job/60928280302?pr=1793, it doesn't seem to.
Is setting nonNullAssertions: false, not doing anything? 😬
Additional Info
❤️🔥
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
package: coreRelated to the core Flint package underlying all other packages and plugins.Related to the core Flint package underlying all other packages and plugins.status: needs investigationFurther research required...? 🔎Further research required...? 🔎type: bugSomething isn't working 🐛Something isn't working 🐛