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

Fix go tests #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix go tests #14

wants to merge 2 commits into from

Conversation

davcamer
Copy link
Contributor

Expected output needed to be updated for the
int 64 as strings vs int 32 as numbers change.

Wrappers was expected for the well known types
but it is one of the ignored files. Referencing
Empty as well means we have one ignored and one
non-ignored file.

Wrapped each dir in t.Run so they will each run
even if one fails.

Expected output needed to be updated for the
int 64 as strings vs int 32 as numbers change.

Wrappers was expected for the well known types
but it is one of the ignored files. Referencing
Empty as well means we have one ignored and one
non-ignored file.

Wrapped each dir in t.Run so they will each run
even if one fails.
@davcamer
Copy link
Contributor Author

closing to re-trigger travis

@davcamer davcamer closed this Aug 12, 2018
@davcamer davcamer reopened this Aug 12, 2018
@davcamer davcamer force-pushed the fix-go-tests branch 3 times, most recently from 4222b87 to a0878fd Compare August 12, 2018 14:14
@davcamer
Copy link
Contributor Author

The travis.yml changes are mostly based on this change, and some follow ups, to the elm-test travis config:
elm-community/elm-test@4588081#diff-354f30a63fb0907d4ad57269548329e3

As described in this elm compiler issue:
elm/compiler#1473
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.

None yet

1 participant