Skip to content
This repository was archived by the owner on Apr 16, 2020. It is now read-only.

Add support for detecting and writing source link files for submodules #229

Closed
wants to merge 2 commits into from
Closed

Conversation

borrrden
Copy link

No description provided.

@borrrden borrrden mentioned this pull request Jul 29, 2017
@ctaggart
Copy link
Owner

ctaggart commented Aug 6, 2017

Thanks @borrrden! I'd like to do a better review then I have time for at the moment. I'm waiting for Visual Studio 15.3 to ship, then there will be a release after that.

@ctaggart ctaggart closed this Aug 18, 2017
@ctaggart ctaggart reopened this Aug 18, 2017
@ctaggart
Copy link
Owner

ctaggart commented Aug 21, 2017

I like the idea of supporting this. I'm going to publish 2.2 without it. I'd like to see it added later on when we have better automated tests and someone has time to do a better review. I also decided to switch back to using a master branch, so I'm going let's do a new pull request. Sorry about the delay.

@ctaggart ctaggart closed this Aug 21, 2017
@borrrden
Copy link
Author

I'm a bit confused by the wording "I'm going let's do." Does that mean you will open the new PR against master or should I?

@ctaggart
Copy link
Owner

Please do a new pull request that targets master and hopefully adds some tests once those are up and running #236.

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

Successfully merging this pull request may close these issues.

2 participants