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

Support Rider properly #3458

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Support Rider properly #3458

wants to merge 5 commits into from

Conversation

@scscgit
Copy link

scscgit commented Jul 17, 2020

Could we have this moved from Global to a root directory, just like VisualStudio.gitignore? For about a year there hasn't been any way to create a JetBrains-compatible project using https://github.com/new due to this removal of Rider compatibility: 96baa63

@scscgit
Copy link

scscgit commented Jul 17, 2020

  • Also is there any plan to support Shared Run Configurations? In Rider it can be like .idea/.idea.NameOfTheProject/.idea/runConfigurations/ with a variable project name (created using "Store as a project file" in the right-top corner of Run Configurations window), while in IDEA it's like .idea/runConfigurations

EDIT: Actually I noticed this is supported implicitly by not being excluded, so I'll instead point out that files under .idea like .name, encodings.xml, indexLayout.xml, or vcs.xml are now being created by default even in a simplest project, even though files like .editorconfig are meant to handle encoding, and the other settings don't have an impact on users who use Visual Studio, while storing their defaults doesn't make any sense, so adding them is far from self-evident, and I would argue against adding them "by default" - they can always be uncommented when needed. Adding so much auto-generated boilerplate by default harms any Rider users for no reason when they join Visual Studio projects. Hopefully there's some compromise to be made, e.g. leaving this "official" way in Global directory, while making a generic VisualStudio or C# .gitignore that removes these files by default. In other words, I think this commit should be simply reverted: 96baa63

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

Successfully merging this pull request may close these issues.

3 participants