Skip to content
This repository was archived by the owner on Dec 16, 2018. It is now read-only.

Conversation

@claudiodekker
Copy link

@claudiodekker claudiodekker commented Dec 11, 2018

This applies the changes made by @zecoj, while also adding a Docker version and significantly adjusting the README file.

Additionally, I've built and pushed the Docker image to the Docker Hub, allowing people to pull it in and use it from there. The relevant command to do so has also been added to the README.

@coveralls
Copy link

coveralls commented Dec 11, 2018

Coverage Status

Coverage remained the same at 98.058% when pulling f616c44 on claudiodekker:master into 7f29bef on cyrozap:master.

Copy link

@rmackinnon rmackinnon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A review of the changes requested all look good, pretty minor stuff. Thank you for taking the time to put together a PR. I'm only a person that's been watching this project, but as a python guy this is a pretty straight forward API endpoint update.

@cyrozap
Copy link
Owner

cyrozap commented Dec 16, 2018

Closing this PR since I'm no longer maintaining this project and will be archiving it soon.

@cyrozap cyrozap closed this Dec 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants