Skip to content

ignore projectSettingsUpdater.xml from Rider #3401

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 2 commits into
base: main
Choose a base branch
from

Conversation

bitbonk
Copy link
Contributor

@bitbonk bitbonk commented May 8, 2020

Reasons for making this change:

This file is generated by Rider and should never be added to VCS.

Links to documentation supporting these rule changes:

To qote @citizenmatt who is one of Rider developers at JetBrains:

This is a file to record when certain settings migrations have occurred. Right now, it only makes changes to VCS configuration, but could be expanded in the future. The VCS changes are to make it the default to show the diff preview in the Commit tool window, and to clear the initial commit message. Once the changes have been made, this file is updated with the current version (“2”) and as long as the version matches, the settings are not re-applied the next time the project is loaded.
It should not be added to source control - it controls defaults to per-user settings ...

@@ -10,6 +10,7 @@

# Generated files
.idea/**/contentModel.xml
.idea/**/projectSettingsUpdater.xml
Copy link

@citizenmatt citizenmatt May 8, 2020

Choose a reason for hiding this comment

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

I'd add a comment that this is a Rider specific file, rather than for all JetBrains IDEs.

You could also make the pattern a little more Rider specific with .idea/*/.idea/projectSettingsUpdater.xml, as Rider has a nested .idea folder to handle multiple solution files in the same folder. (.NET solutions are file based, and other JetBrains IDEs' projects are folder based)

Also simplify it, recursive exclusion is not necessary
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 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 participants