Skip to content

chore: add eslintignore and prettierignore#671

Merged
CodeWithCJ merged 1 commit intoCodeWithCJ:mainfrom
Sim-sat:main
Feb 15, 2026
Merged

chore: add eslintignore and prettierignore#671
CodeWithCJ merged 1 commit intoCodeWithCJ:mainfrom
Sim-sat:main

Conversation

@Sim-sat
Copy link
Contributor

@Sim-sat Sim-sat commented Feb 15, 2026

No description provided.

@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @Sim-sat, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces configuration files for ESLint and Prettier to exclude specific directories from their respective linting and formatting processes. This ensures that only relevant source code is analyzed and formatted, improving performance and preventing conflicts with generated or third-party files.

Highlights

  • ESLint Configuration: Added a .eslintignore file to prevent ESLint from processing build artifacts, coverage reports, distribution files, and node_modules.
  • Prettier Configuration: Added a .prettierignore file to prevent Prettier from formatting build artifacts, coverage reports, distribution files, and node_modules.
Changelog
  • SparkyFitnessFrontend/.eslintignore
    • Added the .eslintignore file.
    • Included build/, coverage/, dist/, and node_modules/ to be ignored by ESLint.
  • SparkyFitnessFrontend/.prettierignore
    • Added the .prettierignore file.
    • Included build/, coverage/, dist/, and node_modules/ to be ignored by Prettier.
Activity
  • No activity recorded for this pull request.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds .eslintignore and .prettierignore files. While the .prettierignore file is appropriate, the .eslintignore file is not the correct approach for this project's configuration. The project uses a modern eslint.config.js (flat config), which does not support .eslintignore files. All ignore patterns should be consolidated within eslint.config.js. I have added a comment with a detailed explanation and a suggested fix.

Comment on lines +1 to +4
build/
coverage/
dist/
node_modules/
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

With the project using a flat configuration file (eslint.config.js), this .eslintignore file is no longer supported and will be ignored by ESLint. The correct way to specify ignored files is within eslint.config.js using the ignores property.

The eslint.config.js file already ignores the dist directory. To ignore the other directories from this file, you should update eslint.config.js:

// eslint.config.js
export default tseslint.config(
  { ignores: ['dist', 'build/', 'coverage/'] },
  // ... rest of the config
);

node_modules is ignored by ESLint by default, so it doesn't need to be explicitly added.

To fix this, please remove the .eslintignore file and update eslint.config.js as shown above. This keeps the configuration centralized and effective.

@CodeWithCJ CodeWithCJ merged commit c6d1bd0 into CodeWithCJ:main Feb 15, 2026
6 checks passed
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