-
Notifications
You must be signed in to change notification settings - Fork 329
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
Bypass diff-so-fancy without breaking pager? (Recommended solution attached) #415
Comments
You can also do Anything that redirects, and thus doesn't require a pager, will be completely bypass d-s-f. Not sure which is best though. |
👍🏻 for the principle though. |
the |
I can confirm the behaviour that @hk0i describes: with
That's what I see on Ubuntu 18, 20 and macOS 10.11. |
@phil-blain That should be a capital If If it's your (What you lose by doing this is most obviously |
I was trying to figure out how to bypass diff-so-fancy for a one-off situation where I wanted a copy/pastable diff format.
The instructions currently recommend:
But my diff was huge so disabling a pager wasn't a very good way to go about it.
I found that the
GIT_PAGER
env variable mentioned inman git
overrides the default pager options as well.I'm opening this ticket to suggest a new recommended way of overriding without breaking paging:
I'd ask everyone to give it a quick try to see if there are any issues with it. If we like it and you guys don't have time I can open a PR later to add it to the pro tips guide.
I figured it would be better to open a ticket first and consult before sending over a random PR.
What do you guys think of this change?
The text was updated successfully, but these errors were encountered: