Unnecessary + 2

Author: JoshuaKGoldberg

Diff for: .all-contributorsrc

@@ -58,8 +58,6 @@ jobs:
       - uses: ./.github/actions/prepare
       - run: pnpm run test --coverage
       - uses: codecov/codecov-action@v3
-        with:
-          flags: unit
   type_check:
     name: Type Check
     runs-on: ubuntu-latest

Diff for: .github/codecov.yml

@@ -1,3 +1,3 @@
-coverage*/
+coverage/
 lib/
 node_modules/

Diff for: .github/workflows/ci.yml

@@ -1,5 +1,5 @@
 .all-contributorsrc
 .husky/
-coverage*/
+coverage/
 lib/
 pnpm-lock.yaml

Diff for: .gitignore

@@ -1,10 +1,9 @@
 {
 	"dictionaries": ["typescript"],
 	"ignorePaths": [
-		"./coverage*",
-		"./script/__snapshots__",
 		".github",
 		"CHANGELOG.md",
+		"coverage",
 		"lib",
 		"node_modules",
 		"pnpm-lock.yaml"
@@ -14,23 +13,24 @@
 		"apexskier",
 		"arethetypeswrong",
 		"automerge",
-		"codecov",
 		"codespace",
 		"contributorsrc",
 		"execa",
 		"infile",
+		"joshuakgoldberg",
 		"knip",
+		"markdownlint",
 		"markdownlintignore",
 		"mtfoley",
 		"npmignore",
 		"npmpackagejsonlintrc",
 		"outro",
 		"packagejson",
-		"precommit",
 		"quickstart",
 		"tada",
 		"tseslint",
 		"tsup",
-		"vitest"
+		"vitest",
+		"wontfix"
 	]
 }

Diff for: .prettierignore

@@ -1,13 +1,3 @@
-/*
-👋 Hi! This ESLint configuration contains a lot more stuff than many repos'!
-You can read from it to see all sorts of linting goodness, but don't worry -
-it's not something you need to exhaustively understand immediately. 💙
-
-If you're interested in learning more, see the 'getting started' docs on:
-- ESLint: https://eslint.org
-- typescript-eslint: https://typescript-eslint.io
-*/
-
 import eslint from "@eslint/js";
 import comments from "@eslint-community/eslint-plugin-eslint-comments/configs";
 import jsdoc from "eslint-plugin-jsdoc";
@@ -56,7 +46,7 @@ export default tseslint.config(
 		languageOptions: {
 			parserOptions: {
 				projectService: {
-					allowDefaultProject: ["*.config.*s", "bin/*.js", "script/*.ts"],
+					allowDefaultProject: ["*.*s", "eslint.config.js"],
 					defaultProject: "./tsconfig.json",
 				},
 				tsconfigRootDir: import.meta.dirname,
@@ -81,21 +71,7 @@ export default tseslint.config(
 			"no-constant-condition": "off",
 
 			// These on-by-default rules work well for this repo if configured
-			"@typescript-eslint/no-unnecessary-condition": [
-				"error",
-				{
-					allowConstantLoopConditions: true,
-				},
-			],
 			"@typescript-eslint/no-unused-vars": ["error", { caughtErrors: "all" }],
-			"@typescript-eslint/prefer-nullish-coalescing": [
-				"error",
-				{ ignorePrimitives: true },
-			],
-			"@typescript-eslint/restrict-template-expressions": [
-				"error",
-				{ allowBoolean: true, allowNullish: true, allowNumber: true },
-			],
 			"n/no-unsupported-features/node-builtins": [
 				"error",
 				{ allowExperimental: true },

Diff for: cspell.json

@@ -1,7 +1,6 @@
 {
 	"$schema": "https://unpkg.com/knip@latest/schema.json",
-	"entry": ["src/index.ts!", "script/*e2e.js"],
-	"ignoreBinaries": ["gh"],
+	"entry": ["src/index.ts!"],
 	"ignoreExportsUsedInFile": { "interface": true, "type": true },
-	"project": ["src/**/*.ts!", "script/**/*.js"]
+	"project": ["src/**/*.ts!"]
 }

Diff for: eslint.config.js

@@ -5,7 +5,7 @@ export function findIntroSectionClose(contents: string) {
 	const projectLogoMatch =
 		/<img align="right" alt="Project logo.+" src=".+">/.exec(contents);
 	if (projectLogoMatch) {
-		return contents.indexOf("\n", projectLogoMatch.index + 2) + 2;
+		return contents.indexOf("\n", projectLogoMatch.index) + 2;
 	}
 
 	// Next: before a first code block or h2, presumably following badges