Skip to content

Add .cache/ to web gitignore #1188

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

Merged

Conversation

dotNomad
Copy link
Collaborator

@dotNomad dotNomad commented Mar 21, 2024

This is an incredibly small change that adds the .cache/ ignore from the root .gitignore to the web/.gitignore. This is so Prettier can read it and avoid formatting the cached Cypress files.

Prettier uses the gitignore:

By default prettier ignores files in version control systems directories (".git", ".sl", ".svn" and ".hg") and node_modules (unless the --with-node-modules CLI option is specified). Prettier will also follow rules specified in the ".gitignore" file if it exists in the same directory from which it is run.

Intent

Part of #1030

Type of Change

    • Bug Fix
    • New Feature
    • Breaking Change
    • Documentation
    • Refactor
    • Tooling

Avoids formatting cypress cache
@dotNomad dotNomad changed the title Add .cache/ to web gitignore Add .cache/ to web gitignore Mar 21, 2024
@dotNomad dotNomad marked this pull request as ready for review March 21, 2024 00:04
Copy link
Collaborator

@sagerb sagerb left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from dotnomad/web-prettier to prettier-introduction-integration-branch March 21, 2024 18:12
@sagerb sagerb merged commit 42f8664 into prettier-introduction-integration-branch Mar 21, 2024
@sagerb sagerb deleted the dotnomad/web-ignore-cache branch March 21, 2024 18:30
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.

4 participants