Skip to content
This repository was archived by the owner on Jun 14, 2024. It is now read-only.

Update .gitignore for IntelliJ on Windows#438

Merged
imback82 merged 1 commit intomicrosoft:masterfrom
sezruby:gitignoreupdate
Apr 29, 2021
Merged

Update .gitignore for IntelliJ on Windows#438
imback82 merged 1 commit intomicrosoft:masterfrom
sezruby:gitignoreupdate

Conversation

@sezruby
Copy link
Copy Markdown
Contributor

@sezruby sezruby commented Apr 29, 2021

What is the context for this pull request?

  • Tracking Issue: n/a
  • Parent Issue: n/a
  • Dependencies: n/a

What changes were proposed in this pull request?

.gitingore update for IntelliJ on windows after #421.
Since IntelliJ git cannot handle symlink properly, files by symlink are shown under "unversioned files":

image

Does this PR introduce any user-facing change?

No

How was this patch tested?

tested with IntelliJ on windows

@sezruby sezruby requested review from clee704 and imback82 April 29, 2021 04:00
@sezruby sezruby self-assigned this Apr 29, 2021
Copy link
Copy Markdown

@clee704 clee704 left a comment

Choose a reason for hiding this comment

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

You should be able to use Git on Windows even if the symlinks are replaced, because we mark those replaced symlinks as --assume-unchanged. I don't like including IDE-specific things in the repo, especially when it's for a workaround that shouldn't be needed, but can't find a better way due to the lack of access to Windows machines.

Copy link
Copy Markdown
Contributor

@imback82 imback82 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @sezruby!

@imback82 imback82 merged commit ba4e712 into microsoft:master Apr 29, 2021
@sezruby sezruby deleted the gitignoreupdate branch April 30, 2021 03:25
@sezruby sezruby added this to the v0.5.0 milestone Jun 10, 2021
@sezruby sezruby added the infrastructure build related, etc. label Jun 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

infrastructure build related, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants