Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

add appveyor for windows ci #96

Merged
merged 6 commits into from
Jan 17, 2017
Merged

add appveyor for windows ci #96

merged 6 commits into from
Jan 17, 2017

Conversation

jessfraz
Copy link
Contributor

#91

testing here

@jessfraz
appveyor
33fed9d 
Signed-off-by: Jess Frazelle <[email protected]>
@jessfraz
fix TestFindRoot() on windows
048e5e4 
Signed-off-by: Jess Frazelle <[email protected]>
@jessfraz
fix rename for windows
e83001b 
Signed-off-by: Jess Frazelle <[email protected]>
@jessfraz
release the source manager in txtWriterTest for windows
41e9e66 
Signed-off-by: Jess Frazelle <[email protected]>
@jessfraz jessfraz mentioned this pull request Jan 14, 2017
Closed
4 tasks
@jessfraz
Copy link
Contributor Author

opened sdboyer/gps#142 but that still doesn't fix the issue, looking more into it.

@jessfraz
Copy link
Contributor Author

jessfraz commented Jan 17, 2017 •
edited
Loading

Ok so here's what I'm going to do:
I can get windows to pass for all the other tests but TestInit() I am going to open an issue documenting the problem and another PR, that turns off the skip on TestInit() but to unblock appveyor and at least get us running the rest of the tests and builds on windows, I think we should merge this.

@jessfraz jessfraz mentioned this pull request Jan 17, 2017
Closed
@jessfraz jessfraz merged commit 11192e4 into golang:master Jan 17, 2017
@jessfraz jessfraz deleted the appveyor branch January 17, 2017 19:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jessfraz @googlebot