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

Remove -o option from diff and commit (fixes #54) #122

Merged
merged 3 commits into from
May 19, 2017

Conversation

techtonik
Copy link
Contributor

No description provided.

@techtonik
Copy link
Contributor Author

Should I break backward compatibility and just fix the tests (clears space for alternative -o usage in future), or spend more time making -o a noop (people scripts continue working).

@spderosso
Copy link
Member

Fix the tests and break backward compatibility (we are still in beta so people shouldn't be scripting around gl).

Thanks!

@techtonik techtonik force-pushed the oless branch 5 times, most recently from 4298807 to 2cb2889 Compare May 19, 2017 15:39
@techtonik
Copy link
Contributor Author

@spderosso ready to merge.

@spderosso spderosso merged commit 9502b46 into gitless-vcs:master May 19, 2017
@techtonik techtonik deleted the oless branch May 19, 2017 16:34
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