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

Removed "core" from .gitignore. #3951

Merged
merged 1 commit into from
Oct 24, 2015
Merged

Conversation

dgquintas
Copy link
Contributor

It was causing git to ignore everything within src/core

@nicolasnoble in case you know how to actually make it work: ignore only core files.

It was causing git to ignore everything within src/core
@sreecha
Copy link
Contributor

sreecha commented Oct 24, 2015

Lgtm. Sorry for the mistake

@dgquintas
Copy link
Contributor Author

No worries, I'm very surprised it behaves this way... there's a nice "git check-ignore" command to "debug" these issues. I wonder what the right way to do it is though.

dgquintas added a commit that referenced this pull request Oct 24, 2015
Removed "core" from .gitignore.
@dgquintas dgquintas merged commit 9698e76 into grpc:master Oct 24, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jan 29, 2019
@lock lock bot unassigned sreecha Jan 29, 2019
@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
CLA GitHub bot

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

Successfully merging this pull request may close these issues.

4 participants