fix!: typo in config

AndreaPontrandolfo · Jul 9, 2024 · 645e220 · 645e220
1 parent 15730a5
commit 645e220
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 14 deletions.
diff --git a/.changeset/lucky-insects-sort.md b/.changeset/lucky-insects-sort.md
@@ -0,0 +1,7 @@
+---
+'eslint-config-sheriff': major
+'@sherifforg/types': major
+'docs-website': patch
+---
+
+fix!: typo in config
diff --git a/apps/docs-website/docs/configuration.mdx b/apps/docs-website/docs/configuration.mdx
@@ -330,18 +330,18 @@ module.exports = defineFlatConfig([...sheriff(sheriffOptions)]);
   </TabItem>
 </Tabs>
 
-### `pathsOveriddes`
+### `pathsOverrides`
 
 As outlined in the [criteria](./core-philosophy/criteria.md) page, Sheriff comes with sensible defaults. However, as your project grows, your team may come across the need to override some of these defaults. This option lets you do just that.
 
 ```ts title="API"
-pathsOveriddes: {
+pathsOverrides: {
   tsconfigLocation: string | string[];
   tests: string[];
 }
 ```
 
-#### `pathsOveriddes.tsconfigLocation`
+#### `pathsOverrides.tsconfigLocation`
 
 By default, Sheriff will use the `project: true` option to locate the `tsconfig.json` of your project.
 
@@ -365,7 +365,7 @@ const sheriffOptions = {
   playwright: false,
   jest: false,
   vitest: false,
-  pathsOveriddes: {
+  pathsOverrides: {
     // highlight-next-line
     tsconfigLocation: "./tsconfig.eslint.json",
   },
@@ -388,7 +388,7 @@ const sheriffOptions = {
   playwright: false,
   jest: false,
   vitest: false,
-  pathsOveriddes: {
+  pathsOverrides: {
     // highlight-next-line
     tsconfigLocation: "./tsconfig.eslint.json",
   },
@@ -400,7 +400,7 @@ module.exports = defineFlatConfig([...sheriff(sheriffOptions)]);
   </TabItem>
 </Tabs>
 
-#### `pathsOveriddes.tests`
+#### `pathsOverrides.tests`
 
 By default, Sheriff will apply Jest or Vitest rules only on specific files.
 
@@ -428,7 +428,7 @@ const sheriffOptions = {
   playwright: false,
   jest: false,
   vitest: false,
-  pathsOveriddes: {
+  pathsOverrides: {
     // highlight-start
     tests: [
       "**/*.mySpecialName.{js,mjs,cjs,ts,mts,cts}",
@@ -456,7 +456,7 @@ const sheriffOptions = {
   playwright: false,
   jest: false,
   vitest: false,
-  pathsOveriddes: {
+  pathsOverrides: {
     // highlight-start
     tests: [
       "**/*.mySpecialName.{js,mjs,cjs,ts,mts,cts,jsx,tsx,mtsx,mjsx}",

diff --git a/apps/docs-website/docs/performance-considerations.mdx b/apps/docs-website/docs/performance-considerations.mdx
@@ -122,5 +122,5 @@ Pay special attention to:
 - [ESLint files and ignore patterns](https://eslint.org/docs/latest/use/configure/configuration-files-new#specifying-files-and-ignores)
 - [Typescript include and exclude patterns](https://typescript-eslint.io/linting/troubleshooting/performance-troubleshooting/#wide-includes-in-your-tsconfig)
 - [Sheriff files options](./configuration#files)
-- [Sheriff pathsOveriddes options](./configuration#pathsoveriddes)
+- [Sheriff pathsOverrides options](./configuration#pathsOverrides)
   - [wide-globs-in-parseroptionsproject](https://typescript-eslint.io/linting/typed-linting/monorepos/#wide-globs-in-parseroptionsproject)
diff --git a/packages/eslint-config-sheriff/src/getBaseConfig.ts b/packages/eslint-config-sheriff/src/getBaseConfig.ts
@@ -27,7 +27,7 @@ import { getBaseEslintHandPickedRules } from './handpickedRules/getBaseEslintHan
 import { getLanguageOptionsTypescript } from './utils/getLanguageOptionsTypescript';
 
 export const getBaseConfig = (userConfigChoices: SheriffSettings) => {
-  const customTSConfigPath = userConfigChoices.pathsOveriddes?.tsconfigLocation;
+  const customTSConfigPath = userConfigChoices.pathsOverrides?.tsconfigLocation;
   const { noRestrictedSyntaxOverride } = userConfigChoices;
   const hasReact = Boolean(userConfigChoices.react);
 

diff --git a/packages/eslint-config-sheriff/src/getExportableConfig.ts b/packages/eslint-config-sheriff/src/getExportableConfig.ts
@@ -30,7 +30,7 @@ export const getExportableConfig = (
     // we insert reactConfig this way because it's an array. It's an array because it contains multiple configs, currently: react, react-hooks, react-a11y and react-refresh.
     exportableConfig = [
       ...exportableConfig,
-      ...getReactConfig(userConfigChoices.pathsOveriddes?.tsconfigLocation),
+      ...getReactConfig(userConfigChoices.pathsOverrides?.tsconfigLocation),
     ];
   }
 
@@ -46,14 +46,14 @@ export const getExportableConfig = (
 
   if (userConfigChoices.jest) {
     exportableConfig.push(
-      getJestConfig(userConfigChoices.pathsOveriddes?.tests),
+      getJestConfig(userConfigChoices.pathsOverrides?.tests),
     );
   }
 
   if (userConfigChoices.vitest) {
     exportableConfig.push(
       //@ts-expect-error
-      getVitestConfig(userConfigChoices.pathsOveriddes?.tests),
+      getVitestConfig(userConfigChoices.pathsOverrides?.tests),
     );
   }
 

diff --git a/packages/sheriff-types/src/index.ts b/packages/sheriff-types/src/index.ts
@@ -89,7 +89,7 @@ export interface SheriffSettings extends Partial<SheriffConfigurablePlugins> {
   /**
    * This parameter allows you to override the paths for some Sheriff settings.
    */
-  pathsOveriddes?: {
+  pathsOverrides?: {
     /**
      * With this setting, if you have multiple tsconfig.json files in your project (like tsconfig.json, tsconfig.eslint.json, tsconfig.node.json, etc...) you can specify which config Sheriff will pickup. You can also specify a list of paths, see: https://typescript-eslint.io/linting/typed-linting/monorepos/#one-tsconfigjson-per-package-and-an-optional-one-in-the-root.
      */