Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring for ESLint and Prettier Integration #486

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Fazleiw
Copy link

@Fazleiw Fazleiw commented Oct 23, 2024

###Description
Refactored the ESLint configuration to include Prettier integration. Added plugin:prettier/recommended and the prettier/prettier rule. Updated the Next.js root directory setting for apps.

###Motivation & Context
This change ensures Prettier formatting is enforced by ESLint and improves compatibility with Next.js projects using multiple apps.
while learning the code quality and project structure

###Code Review
Check that ESLint and Prettier work together without conflicts.
Confirm the rootDir setting works as expected for the Next.js apps.

###Type of Change
Refactor

###Checklist
I have done a self-review of my own code.
Any new and existing tests pass locally with my changes.
My changes generate no new warnings.

By adding the plugin:prettier/recommended and prettier/prettier rules,
improved the configuration to ensure that Prettier formatting issues are handled by ESLint
Copy link

vercel bot commented Oct 23, 2024

@Fazleiw is attempting to deploy a commit to the Nouns Builder Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant