-
Notifications
You must be signed in to change notification settings - Fork 8
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
Bugfix/get server version gocd 19.5.0 #13
base: master
Are you sure you want to change the base?
Bugfix/get server version gocd 19.5.0 #13
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't claim I have carefully reviewed the code but I did some testing with gocd 19.5.0 and it seems to work fine (so far).
The tests haven't been touched in a long time, that shows. There will be a separate PR fixing the tests. |
#15 Should be merged first |
Include tfbogdan's sugestion Co-Authored-By: bogdan tudoran <tfbogdan@gmail.com>
0af423e
to
3736630
Compare
So what should we do about this merge request? |
I am still having trouble getting the tests to run, did not have too much time lately to look into it. The fix is functional, as far as I/we can tell. |
Fixes #12