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

Updated specgen tool to use a go module and added get service logs api call #472

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

The-TT-Hacker
Copy link

Updated the specgen tool so it uses the newer go modules instead of godep.
I also updated the docker version imported to the latest 19.03.13.

This was tested on a fresh install of go on a new machine. All you have to do to update the models is install go, update the docker repo which is imported using go get -u github.com/docker/docker@ and then run the update-generate-code.cmd script. It downloads all of the correct dependencies, builds the program and generates the models. This has been updated in the readme for specgen.

I have also added a call to the get service logs endpoint. To do this I had to add a model which was the reason for updating the go specgen program.

@ACoderLife
Copy link
Contributor

This pull request can be closed when @589 is in.
The important backend changes were done in #480

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

Successfully merging this pull request may close these issues.

2 participants