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

add test run on windows to ci #106

Merged
merged 7 commits into from
Nov 4, 2020
Merged

add test run on windows to ci #106

merged 7 commits into from
Nov 4, 2020

Conversation

dron22
Copy link
Contributor

@dron22 dron22 commented Oct 30, 2020

This PR adds a test run via github actions on a windows server to ci.

@gandarez Some tests are failing, expecially in project package. As you also work on a windows machine, can you fix these and push commits to this PR?

@gandarez gandarez force-pushed the feature/ci-windows-tests branch from 2b52fe9 to 0549993 Compare November 3, 2020 18:43
@gandarez gandarez added the p0 High Priority label Nov 3, 2020
@gandarez gandarez added this to the v0.0.1 milestone Nov 3, 2020
@gandarez gandarez force-pushed the feature/ci-windows-tests branch 4 times, most recently from d4447c2 to 9b48547 Compare November 4, 2020 00:11
@gandarez gandarez force-pushed the feature/ci-windows-tests branch from 9b48547 to 8f1ff94 Compare November 4, 2020 00:20
@gandarez
Copy link
Member

gandarez commented Nov 4, 2020

Fixed all tests for Windows. The moral is not to use path.* and instead use filepath.* to get it working as expected for Windows devices. Also added some minor adjustments. golang/go#1423

I will fix linter installation.

@gandarez
Copy link
Member

gandarez commented Nov 4, 2020

@andy linting fixed.

@dron22
Copy link
Contributor Author

dron22 commented Nov 4, 2020

LGTM 👍

@gandarez gandarez merged commit 241708f into master Nov 4, 2020
@gandarez gandarez deleted the feature/ci-windows-tests branch November 4, 2020 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p0 High Priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants