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

Sort missing parameters in the failure message #14

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@choroba
Contributor

choroba commented Apr 26, 2017

to get consistent output each time. Fixes 115241.

@autarch

This comment has been minimized.

Show comment
Hide comment
@autarch

autarch Apr 26, 2017

Collaborator

This is failing in Travis because of something I broke. You should rebase against master to get the fix I just pushed.

That said, the changes look good. Thanks!

Collaborator

autarch commented Apr 26, 2017

This is failing in Travis because of something I broke. You should rebase against master to get the fix I just pushed.

That said, the changes look good. Thanks!

Sort missing parameters in the failure message
to get consistent output each time. Fixes 115241.
@choroba

This comment has been minimized.

Show comment
Hide comment
@choroba

choroba Apr 26, 2017

Contributor

☑️ rebased

Contributor

choroba commented Apr 26, 2017

☑️ rebased

@autarch

This comment has been minimized.

Show comment
Hide comment
@autarch

autarch Apr 29, 2017

Collaborator

Merged from the CLI. Thanks!

Collaborator

autarch commented Apr 29, 2017

Merged from the CLI. Thanks!

@autarch autarch closed this Apr 29, 2017

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