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

Create a CI recipe for the project #56

Closed
diogoosorio opened this issue Oct 5, 2017 · 4 comments
Closed

Create a CI recipe for the project #56

diogoosorio opened this issue Oct 5, 2017 · 4 comments

Comments

@diogoosorio
Copy link
Contributor

I've just opened a PR that introduces some tests to the project.

The next step would be to make sure that the test battery is run every time a PR is opened. A CI (continuous integration) system is great for automating this kind of job.

Tracis CI appears to be the de-facto service for this kind of work in the Github/open source community.

I'm up for writing the recipe for the system (for now to run the test battery each time a PR is opened), but the task would then require for a maintainer to open up an account on Travis CI (so that it would be associated with this repository).

Is this something worth pursuing? What are your thoughts about this?

@codingo
Copy link
Owner

codingo commented Oct 5, 2017

@diogoosorio Definitely interested in this. I've linked this on Travis CI and @ufocoder has done some work writing this up but it looks to be in error at the moment (see #58 for log)

Definitely would love if you can do the final push for this!

@codingo
Copy link
Owner

codingo commented Oct 6, 2017

#58 is now resolved with #59 but would definitely appreciate assistance wiring in your test cases!

@diogoosorio
Copy link
Contributor Author

@ufocoder did include running the test suite as one of the integration validations. You can see them running on this job, for example.

I'm going to close this issue. 🎉

@codingo
Copy link
Owner

codingo commented Oct 8, 2017

My sincere apologies @ufocoder, this is definitely my bad. I'm still quite new to continuous integration cases within python. Appreciate all that you both do for this project!

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

No branches or pull requests

2 participants