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

chore: Remove .idea files #27

Merged
merged 1 commit into from
Nov 7, 2023
Merged

chore: Remove .idea files #27

merged 1 commit into from
Nov 7, 2023

Conversation

WillBAnders
Copy link
Member

These files were added in #11 and should not have been committed. Using a .gitignore can help with this, though my personal "hot take" is that .gitignore can be a crutch and it's better to pay close attention to what files are being committed and never rely on git add . (especially when working across repositories, as the .gitignore configuration may vary). YMMV.

@WillBAnders WillBAnders requested a review from a team October 31, 2023 22:41
@WillBAnders WillBAnders self-assigned this Oct 31, 2023
Copy link
Contributor

@Brian-Magnuson Brian-Magnuson left a comment

Choose a reason for hiding this comment

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

I personally prefer adding to the .gitignore and I think it would be good for this project. Only thing is that I don't have nearly as much experience with Git as you, @WillBAnders, so maybe you know best. I'll approve these changes, so you can merge if you want to.

@WillBAnders WillBAnders merged commit 47f701e into master Nov 7, 2023
2 checks passed
@WillBAnders WillBAnders deleted the chore/remove-idea-files branch November 7, 2023 22:46
@WillBAnders
Copy link
Member Author

@Brian-Magnuson Mentioned on Discord; I'm not a fan personally but if y'all want to set one up feel free. Either way, these files need to be removed from the project so this has to be merged.

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