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

Ignore patterns from global gitignore #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matusf
Copy link

@matusf matusf commented Mar 18, 2020

Hi! First of all, thanks for this awesome extension!
I've added a functionality to ignore patterns from global git ignore. I'm sorry if this is not the most idiomatic typescript, I was coding in typescript for the first time 😃 However, I'm happy to follow your review recommendations if you have any.


closes #1

@npx
Copy link
Owner

npx commented Mar 21, 2020

Hello @matusf!

Thank you for your work!
In order to incorportate your change more easily I updated the project a little bit and added code formatting.

I would appreciate it if you could rebase your changes onto my latest upstream changes.

As for the code, I will add some comments!

@matusf matusf force-pushed the 1-include-global-gitignore branch from e7bdc1b to cb4dc01 Compare April 2, 2020 18:50
@matusf
Copy link
Author

matusf commented Apr 2, 2020

@npx rebased!

@npx
Copy link
Owner

npx commented Apr 2, 2020

@matusf can you see the comments I made a while ago?

@matusf
Copy link
Author

matusf commented Apr 2, 2020

No, I don't see any of them.

@matusf
Copy link
Author

matusf commented Jan 6, 2021

ping @npx

Copy link
Owner

@npx npx left a comment

Choose a reason for hiding this comment

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

Does this make it visible? Can you see the review now? @matusf

src/gitignore-hider.ts Outdated Show resolved Hide resolved
src/gitignore-hider.ts Outdated Show resolved Hide resolved
src/gitignore-reader.ts Outdated Show resolved Hide resolved
src/gitignore-hider.ts Show resolved Hide resolved
@matusf
Copy link
Author

matusf commented Jan 12, 2021

Yes, I see it now, thanks. I'll make the changes shortly.

@matusf matusf requested a review from npx January 13, 2021 12:49
- add function for expanding paths starting with `~`
- add method for getting path to global gitignore from git config
@matusf
Copy link
Author

matusf commented Feb 11, 2021

Hi @npx, I've made the changes. Could you please check if it's all ok?

@LuisPeregrina
Copy link

@npx good enhancement. I would vote to include this.

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

Successfully merging this pull request may close these issues.

Global .gitignore could be included as well.
3 participants