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

Fix opening editor from diff-stat line #783

Closed
wants to merge 1 commit into from
Closed

Fix opening editor from diff-stat line #783

wants to merge 1 commit into from

Conversation

h3xx
Copy link
Contributor

@h3xx h3xx commented Feb 13, 2018

Partial fix for #781

This still doesn't fix the underlying issue where the working directory gets set incorrectly when in a submodule.

Partial fix for #781

This still doesn't fix the issue where the working directory gets set
incorrectly when in a submodule.
@jonas
Copy link
Owner

jonas commented Mar 13, 2018

Thanks a lot. It would be nice to have a test for this since sub modules are not “well” supported. I will try to find some time.

@h3xx
Copy link
Contributor Author

h3xx commented Mar 16, 2018

See f99d2af for a related unit test (not in this branch for some reason).

jonas added a commit that referenced this pull request May 20, 2018
Add tests for simple use of submodules and worktrees together with
editors from subdirectories.

Closes #783
Closes #508
@jonas jonas closed this in 625bb58 May 20, 2018
@h3xx
Copy link
Contributor Author

h3xx commented May 21, 2018

Confirmed fixed. Thanks!

rolandwalker pushed a commit to rolandwalker/tig that referenced this pull request May 21, 2018
Add tests for simple use of submodules and worktrees together with
editors from subdirectories.

Closes jonas#783
Closes jonas#508
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.

2 participants