Skip to content
This repository has been archived by the owner on Nov 18, 2021. It is now read-only.

start using GitHub Actions as CI #147

Closed
wants to merge 1 commit into from
Closed

start using GitHub Actions as CI #147

wants to merge 1 commit into from

Conversation

mvdan
Copy link
Contributor

@mvdan mvdan commented Oct 6, 2019

For now, this tests on Ubuntu, MacOS, and Windows, both on the latest Go
1.12 and 1.13. We run the tests both with and without -race.

Three tests needed updating, since they exec the system's "echo" binary.
On Windows, that may use CRLF line endings, which happened on GitHub
Actions. To fix that, match the output with a line regular expression,
instead of via an entire stdout file with LF line endings.

Finally, remove the AppVeyor badge and cloudbuild file, as we won't be
needing those anymore.

Fixes #144.

For now, this tests on Ubuntu, MacOS, and Windows, both on the latest Go
1.12 and 1.13. We run the tests both with and without -race.

Three tests needed updating, since they exec the system's "echo" binary.
On Windows, that may use CRLF line endings, which happened on GitHub
Actions. To fix that, match the output with a line regular expression,
instead of via an entire stdout file with LF line endings.

Finally, remove the AppVeyor badge and cloudbuild file, as we won't be
needing those anymore.

Fixes #144.
@mvdan
Copy link
Contributor Author

mvdan commented Oct 6, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consolidate CI on GitHub Actions
2 participants