Skip to content

Support negated .gitignore rules.#73890

Merged
jrieken merged 1 commit intomicrosoft:masterfrom
markspanbroek:master
May 24, 2019
Merged

Support negated .gitignore rules.#73890
jrieken merged 1 commit intomicrosoft:masterfrom
markspanbroek:master

Conversation

@markspanbroek
Copy link
Contributor

Fixes issues #69035 and #38112.

When invoking git check-ignore it returns those paths that match against a .gitignore pattern, whether it's a negated pattern or not. This change ensures that negated pattern matches do not mark a file as ignored.

@msftclas
Copy link

msftclas commented May 17, 2019

CLA assistant check
All CLA requirements met.

@joaomoreno
Copy link
Member

@jrieken Did you try this when implementing the ignore markers?

@joaomoreno joaomoreno added this to the Backlog milestone May 23, 2019
@joaomoreno joaomoreno added the git GIT issues label May 23, 2019
@jrieken
Copy link
Member

jrieken commented May 23, 2019

@jrieken Did you try this when implementing the ignore markers?

Nope, but @markspanbroek seems to know this better, the doc actually says this

@joaomoreno joaomoreno removed the request for review from jrieken May 23, 2019 13:37
@jrieken
Copy link
Member

jrieken commented May 24, 2019

This actually isn't true on my system (using git 2.17) @markspanbroek Is this a recent fix/feature in git?

Screenshot 2019-05-24 at 12 26 11

@jrieken jrieken added the info-needed Issue requires more information from poster label May 24, 2019
@markspanbroek
Copy link
Contributor Author

@jrieken it requires the -v flag, I also added this to line 1159 of repository.ts.

@jrieken
Copy link
Member

jrieken commented May 24, 2019

@jrieken it requires the -v flag, I also added this to line 1159 of repository.ts.

Thanks - didn't see and know that. The power of open source! @joaomoreno I am 👍 for this

@jrieken jrieken self-requested a review May 24, 2019 13:17
@jrieken jrieken modified the milestones: Backlog, May 2019 May 24, 2019
@jrieken jrieken removed the info-needed Issue requires more information from poster label May 24, 2019
@jrieken jrieken merged commit 4976cff into microsoft:master May 24, 2019
@jrieken
Copy link
Member

jrieken commented May 24, 2019

This will be in next insiders and stable - Thanks for all the help!

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

git GIT issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants