Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 30, 2025

Consolidates 8 scattered ESLint configuration files into a single .eslintrc.json at the repository root to enforce consistent code style across all packages and tests.

Changes

  • Created root .eslintrc.json with unified rules for the entire monorepo
  • Removed 7 duplicate configs from packages/*/src/.eslintrc.json
  • Removed tests/.eslintrc.json (was extending non-existent parent config)
  • Added override section to preserve no-console: off for test files (**/*.test.ts, **/*.spec.ts, tests/**/*.ts)
  • Added jest: true to env for universal test environment support

Configuration

{
  "root": true,
  "parser": "@typescript-eslint/parser",
  "rules": {
    "@typescript-eslint/no-unused-vars": ["error", { "argsIgnorePattern": "^_" }],
    "prefer-const": "error",
    "no-console": "warn",
    "semi": ["error", "always"],
    "quotes": ["error", "single"]
  },
  "overrides": [
    {
      "files": ["tests/**/*.ts", "**/*.test.ts", "**/*.spec.ts"],
      "rules": { "no-console": "off" }
    }
  ]
}

Packages can still extend or override specific rules as needed via their own .eslintrc.json files.

Original prompt

This section details on the original issue you should resolve

<issue_title>Move all ESLint configuration to a centralized location at the root of the repo</issue_title>
<issue_description>We should want all packages and tests to follow the same ESLint rules. Right now, there are ESLint configuration files all over the place. We should remove them and have a single one for the entire repository. If needed each package can customize small things, but in general I'd like the repository to follow a cohesive coding style.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@juliomenendez
Copy link

juliomenendez commented Oct 30, 2025

@copilot We need to enable a step on the CI to check the styles using ESLint. Also the ESLint should cover the entire repository.

Copilot AI and others added 2 commits October 30, 2025 23:29
Co-authored-by: juliomenendez <9697+juliomenendez@users.noreply.github.com>
Co-authored-by: juliomenendez <9697+juliomenendez@users.noreply.github.com>
Copilot AI changed the title [WIP] Move all ESLint configuration to root of the repository Centralize ESLint configuration to repository root Oct 30, 2025
Copilot AI requested a review from juliomenendez October 30, 2025 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move all ESLint configuration to a centralized location at the root of the repo

2 participants