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
Andy/ci compatibility #485
Conversation
go/cmd/dolt/commands/status.go
Outdated
@@ -97,7 +97,7 @@ func (cmd StatusCmd) Exec(ctx context.Context, commandStr string, args []string, | |||
} | |||
|
|||
printStatus(ctx, dEnv, stagedTblDiffs, notStagedTblDiffs, workingTblsInConflict, workingDocsInConflict, stagedDocDiffs, notStagedDocDiffs) | |||
return 0 | |||
return 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@reltuk I added this commit to fail the compatibility tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was going to be a lot cleaner, but then the windows tests failed and I had to rebase off of master. Please see build #2 for confirmation that exit codes work correctly.
ab91e9c
to
9b4d2be
Compare
335d928
to
b6b0506
Compare
e8ed89b
to
994a30b
Compare
… testing; cleanup test setup for forwards/backwards compatibility testing.
No description provided.