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

mrview: fix -norealign option #1473

Merged
merged 1 commit into from Oct 23, 2018

Conversation

Projects
None yet
2 participants
@jdtournier
Member

jdtournier commented Oct 22, 2018

Processing of this option was removed in merge commit 42573be, presumably due to a mistake during conflict resolution.

Suggest merging to master directly since this is a clear bug fix, and the fix is totally benign.

[EDIT]
See original report on the forum.

mrview: fix -norealign option
Processing of this option was removed in merge commit 42573be,
presumably due to a mistake during conflict resolution.

@jdtournier jdtournier self-assigned this Oct 22, 2018

@thijsdhollander

Yeah, that definitely looks straightforward. Fully agreed to have it go immediately to master. 👍

@jdtournier jdtournier merged commit 16090b5 into master Oct 23, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jdtournier jdtournier deleted the fix_mrview_norealign_option branch Oct 23, 2018

@thijsdhollander thijsdhollander restored the fix_mrview_norealign_option branch Nov 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment