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

Remove Appveyor, replace Travis with CircleCI #262

Closed
yanokwa opened this issue Feb 11, 2019 · 3 comments
Closed

Remove Appveyor, replace Travis with CircleCI #262

yanokwa opened this issue Feb 11, 2019 · 3 comments

Comments

@yanokwa
Copy link
Contributor

yanokwa commented Feb 11, 2019

My guess is that Appveyor was used to verify Windows support, but pyxform is pretty platform agnostic, so I don't think think it's needed.

Travis works fine, but the ODK tools all use CircleCI and I think some consistency there would be nice. One tricky thing here will be running the tests on the various versions of Python.

@ukanga
Copy link
Contributor

ukanga commented Feb 15, 2019

In terms of making pyxform ODK tools compliant I agree. However, where windows is concerned, we introduced AppVeyor after conversations in this PR #116 necessitated it. I deploy pyxform only on Linux environments but that is not the case for all pyxform users. Can this be reconsidered?

@yanokwa
Copy link
Contributor Author

yanokwa commented Feb 15, 2019

Seems like the core of the issue is the install (See #63 (comment)). Agreed that we should keep Appveyor, but maybe replace Travis with CircleCI? I've updated the PR so you see what that looks like.

@ukanga
Copy link
Contributor

ukanga commented Feb 19, 2019

... replace Travis with CircleCI?

Ok with me.

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

No branches or pull requests

2 participants