Sitelet https://web.archive.org/web/20210210085053/https://github.com/github/gitignore/pull/3401

github / gitignore

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

Jump to bottom

ignore projectSettingsUpdater.xml from Rider #3401

Open

bitbonk wants to merge 2 commits into github:master from bitbonk:patch-2

Contributor

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 ...


          ignore projectSettingsUpdater.xml from Rider

8bfc96e

citizenmatt reviewed

Global/JetBrains.gitignore Outdated

  
      @@ -10,6 +10,7 @@
    
        # Generated files

        .idea/**/contentModel.xml

        .idea/**/projectSettingsUpdater.xml

citizenmatt May 8, 2020 •

edited

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)


          Move projectSettingsUpdater.xml to Rider section

5cca992

Also simplify it, recursive exclusion is not necessary

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