Skip to content

chore(eslint): migrate to flat configuration#131

Merged
minuukang merged 4 commits intotoss:mainfrom
sukvvon:chore/migrate-eslint-to-flat-config
Apr 23, 2025
Merged

chore(eslint): migrate to flat configuration#131
minuukang merged 4 commits intotoss:mainfrom
sukvvon:chore/migrate-eslint-to-flat-config

Conversation

@sukvvon
Copy link
Contributor

@sukvvon sukvvon commented Mar 17, 2025

  • Migrate to falt configuration
    • To maintain the existing file format, added '@typescript-eslint/no-empty-interface': 'off'.
    • Removed prettier/prettier: 'warn' since eslint-plugin-prettier already handles this.
  • To maintain the existing file format, added eslint-disable @typescript-eslint/no-require-imports in docs/next.config.js and packages/next/vitestSetup.ts.

@sukvvon sukvvon marked this pull request as draft March 17, 2025 12:25
@sukvvon sukvvon force-pushed the chore/migrate-eslint-to-flat-config branch from e37e498 to f216751 Compare March 17, 2025 12:34
@sukvvon sukvvon marked this pull request as ready for review March 17, 2025 12:36
@sukvvon sukvvon changed the title chore(eslint): migrate to flat config chore(eslint): migrate to flat configuration Mar 17, 2025
@sukvvon
Copy link
Contributor Author

sukvvon commented Apr 7, 2025

@minuukang

I recognized that pnpm-lock.yaml was entirely updated to the latest versions instead of the versions specified in package.json, and I modified it so that only eslint-related libraries are updated.

Copy link
Member

@minuukang minuukang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks to contribute. I'm so late to check this 🙇🏽

@minuukang minuukang merged commit 6435ffc into toss:main Apr 23, 2025
1 check passed
@sukvvon sukvvon deleted the chore/migrate-eslint-to-flat-config branch April 23, 2025 04:31
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.

2 participants

Comments