Skip to content

chore: update git and prettier ignores#38

Merged
G-Rath merged 5 commits into
mainfrom
cleanup-ignores
Apr 28, 2025
Merged

chore: update git and prettier ignores#38
G-Rath merged 5 commits into
mainfrom
cleanup-ignores

Conversation

@G-Rath

@G-Rath G-Rath commented Apr 25, 2025

Copy link
Copy Markdown
Collaborator

These are generally all pretty standard stuff, though I had not realized __pycache__ was a thing that Python generates as it turns out my IDE magically hides it from me.

I've included the tool caches even though they actually include a .gitignore with a wildcard since that doesn't stop prettier from checking the directories and I think it's nicer to be consistent between the two files. I've also included node_modules and vendor since they're traditional and are usually very big so I think it's nice having them as a just-in-case

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • .prettierignore: Language not supported

@G-Rath

G-Rath commented Apr 25, 2025

Copy link
Copy Markdown
Collaborator Author

Fun fact, this actually takes about 5 seconds off prettier locally for me, as I've got a lot of stuff shelved in the .idea directory 😅

@Unifex Unifex left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@G-Rath G-Rath merged commit d5c69af into main Apr 28, 2025
@G-Rath G-Rath deleted the cleanup-ignores branch April 28, 2025 03:55
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.

5 participants