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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dependencies #265

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

Conversation

fritterhoff
Copy link

Hi,

several dependencies are a bit outdated so I've updated them (including the changes for #220 ). According to my tests nothing should be broken afterwards 馃槈

@fritterhoff
Copy link
Author

Hi @joohoi,

is the project still maintained?

Thanks :)

@joohoi
Copy link
Owner

joohoi commented Jun 21, 2021

Hi, It is. I have just been really busy lately! Thanks for the PR, I'll look into it within a few days.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 72.805% when pulling 878ea31 on fritterhoff:feat-dependencies into 68bb6ab on joohoi:master.

@fritterhoff
Copy link
Author

Hi, @joohoi do you have any update for me? 馃榾

@fritterhoff
Copy link
Author

Hello? :)
Do you need any further information/tests/... ?
Thanks 馃憤馃徏

@ZPascal
Copy link

ZPascal commented Aug 22, 2021

Hi @joohoi could you please check the PR? I think it makes sense to update the Golang version (if possible). @fritterhoff I think in the meantime, it could make sense to update to version 1.17 directly, what do you think?

@fritterhoff
Copy link
Author

Sure - it makes more sense to use an up-to-date version, but without any feedback/reaction it is a pity to waste any time 馃槥

@fritterhoff
Copy link
Author

@ZPascal I've updated the dependencies and the base image to 1.17 a few days ago.

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.

None yet

4 participants