Merge pull request #302 from noaignite/feat/eslint-config-allow-dev-d…

…eps-in-tsup-config

feat(eslint-config): allow dev deps in tsup config

.changeset/strong-boxes-heal.md

@@ -0,0 +1,2 @@
+---
+---

packages/create-app/template-eslint-config/library.cjs

@@ -42,12 +42,14 @@ module.exports = {
  'import/no-extraneous-dependencies': [
  'error',
  {
- // Allow imports of `devDependencies` inside test files.
+ // Allow imports of `devDependencies` inside certain files.
  devDependencies: [
  'test.{ts,tsx}', // repos with a single test file
  'test-*.{ts,tsx}', // repos with multiple top-level test files
  '**/*{.,_}{test,spec}.{ts,tsx}', // tests where the extension or filename suffix denotes that it is a test
- '**/vitest.config.mts', // vitest config
+ '**/scripts/**/*.{mts,ts,tsx}', // script files
+ '**/tsup.config.{mts,ts,tsx}', // tsup files
+ '**/vitest.config.{mts,ts,tsx}', // vitest files
  ],
  optionalDependencies: false,
  },
@@ -58,5 +60,6 @@ module.exports = {
 
  'lines-around-directive': ['error', 'always'], // Enable & let prettier format.
  'no-console': ['error', { allow: ['info', 'warn', 'error'] }], // Allow warn and error logs.
+ 'prefer-named-capture-group': 'off', // Too strict.
  },
 }

packages/create-app/template-eslint-config/next.cjs

@@ -47,12 +47,14 @@ module.exports = {
  'import/no-extraneous-dependencies': [
  'error',
  {
- // Allow imports of `devDependencies` inside test files.
+ // Allow imports of `devDependencies` inside certain files.
  devDependencies: [
  'test.{ts,tsx}', // repos with a single test file
  'test-*.{ts,tsx}', // repos with multiple top-level test files
  '**/*{.,_}{test,spec}.{ts,tsx}', // tests where the extension or filename suffix denotes that it is a test
- '**/vitest.config.mts', // vitest config
+ '**/scripts/**/*.{mts,ts,tsx}', // script files
+ '**/tsup.config.{mts,ts,tsx}', // tsup files
+ '**/vitest.config.{mts,ts,tsx}', // vitest files
  ],
  optionalDependencies: false,
  },
@@ -67,5 +69,6 @@ module.exports = {
 
  'lines-around-directive': ['error', 'always'], // Enable & let prettier format.
  'no-console': ['error', { allow: ['info', 'warn', 'error'] }], // Allow warn and error logs.
+ 'prefer-named-capture-group': 'off', // Too strict.
  },
 }

packages/create-app/template-eslint-config/react.cjs

@@ -44,12 +44,14 @@ module.exports = {
  'import/no-extraneous-dependencies': [
  'error',
  {
- // Allow imports of `devDependencies` inside test files.
+ // Allow imports of `devDependencies` inside certain files.
  devDependencies: [
  'test.{ts,tsx}', // repos with a single test file
  'test-*.{ts,tsx}', // repos with multiple top-level test files
  '**/*{.,_}{test,spec}.{ts,tsx}', // tests where the extension or filename suffix denotes that it is a test
- '**/vitest.config.mts', // vitest config
+ '**/scripts/**/*.{mts,ts,tsx}', // script files
+ '**/tsup.config.{mts,ts,tsx}', // tsup files
+ '**/vitest.config.{mts,ts,tsx}', // vitest files
  ],
  optionalDependencies: false,
  },
@@ -64,6 +66,7 @@ module.exports = {
 
  'lines-around-directive': ['error', 'always'], // Enable & let prettier format.
  'no-console': ['error', { allow: ['info', 'warn', 'error'] }], // Allow warn and error logs.
+ 'prefer-named-capture-group': 'off', // Too strict.
  },
  overrides: [
  {

packages/eslint-config/library.cjs

@@ -42,13 +42,14 @@ module.exports = {
  'import/no-extraneous-dependencies': [
  'error',
  {
- // Allow imports of `devDependencies` inside test files.
+ // Allow imports of `devDependencies` inside certain files.
  devDependencies: [
  'test.{ts,tsx}', // repos with a single test file
  'test-*.{ts,tsx}', // repos with multiple top-level test files
  '**/*{.,_}{test,spec}.{ts,tsx}', // tests where the extension or filename suffix denotes that it is a test
- '**/vitest.config.mts', // vitest config
- '**/scripts/**/*.{ts,tsx}', // build script files
+ '**/scripts/**/*.{mts,ts,tsx}', // script files
+ '**/tsup.config.{mts,ts,tsx}', // tsup files
+ '**/vitest.config.{mts,ts,tsx}', // vitest files
  ],
  optionalDependencies: false,
  },

packages/eslint-config/next.cjs

@@ -47,13 +47,14 @@ module.exports = {
  'import/no-extraneous-dependencies': [
  'error',
  {
- // Allow imports of `devDependencies` inside test files.
+ // Allow imports of `devDependencies` inside certain files.
  devDependencies: [
  'test.{ts,tsx}', // repos with a single test file
  'test-*.{ts,tsx}', // repos with multiple top-level test files
  '**/*{.,_}{test,spec}.{ts,tsx}', // tests where the extension or filename suffix denotes that it is a test
- '**/vitest.config.mts', // vitest config
- '**/scripts/**/*.{ts,tsx}', // build script files
+ '**/scripts/**/*.{mts,ts,tsx}', // script files
+ '**/tsup.config.{mts,ts,tsx}', // tsup files
+ '**/vitest.config.{mts,ts,tsx}', // vitest files
  ],
  optionalDependencies: false,
  },

packages/eslint-config/react.cjs

@@ -44,13 +44,14 @@ module.exports = {
  'import/no-extraneous-dependencies': [
  'error',
  {
- // Allow imports of `devDependencies` inside test files.
+ // Allow imports of `devDependencies` inside certain files.
  devDependencies: [
  'test.{ts,tsx}', // repos with a single test file
  'test-*.{ts,tsx}', // repos with multiple top-level test files
  '**/*{.,_}{test,spec}.{ts,tsx}', // tests where the extension or filename suffix denotes that it is a test
- '**/vitest.config.mts', // vitest config
- '**/scripts/**/*.{ts,tsx}', // build script files
+ '**/scripts/**/*.{mts,ts,tsx}', // script files
+ '**/tsup.config.{mts,ts,tsx}', // tsup files
+ '**/vitest.config.{mts,ts,tsx}', // vitest files
  ],
  optionalDependencies: false,
  },