Skip to content

Update Unity.gitignore - Added ignore for JetBrains Rider #3543

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

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

Conversation

MickL
Copy link

@MickL MickL commented Oct 9, 2020

Reasons for making this change:
Rider from JetBrains is a commonly used C# IDE. It will add a .idea folder which should be ignored in git.

Added ignore for JetBrains Rider
@thatrobotdev
Copy link
Contributor

Copy link

This PR is stale because there have been no updates in 90 days. It will close after 180 days of inactivity. Leave a comment if you want to keep it open 😄

@github-actions github-actions bot added the stale label May 31, 2025
@MickL
Copy link
Author

MickL commented May 31, 2025

Yeah lets close unresolved issues, thats how to solve things the right way!

@thorrsson
Copy link
Collaborator

Hey @MickL
In case you missed the announcement, we added Stalebot to the repo to help prune down the hundreds of PRs that had been left unattended for as long as they had been.

As noted in the post, just adding a comment to the PR to identify it remains relevant is all you need to do to get it on our backlog.

@thorrsson
Copy link
Collaborator

I'd point out the Contribution Guidelines https://github.com/github/gitignore?tab=readme-ov-file#contributing-guidelines (We do have a rewrite of this coming but the guidance for Tool specific exclusions in a Language file will likely remain the same)

Specifically

Please consider the scope of your change. If your change is specific to a certain language or framework, then make sure the change is made to the template for that language or framework, rather than to the template for an editor, tool, or operating system.

Since the .idea folder isn't specific to Unity, nor Unity development I don't think this change belongs in this file. The .idea directory also stores Shared Run Configurations and some other artifacts that folks benefit from including in their committed changes. Ignoring this entire directory shouldn't be the standard, but optional for teams who choose not to include it.

** I acknowledge the .vs line in this file as well... it should not have been pushed through in the first place if we were to apply the Contribution guidelines **

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 participants