Skip to content

Conversation

guggero
Copy link
Contributor

@guggero guggero commented Jul 12, 2022

@Roasbeef could you please add the DOCKER_USERNAME and DOCKER_API_KEY environment variables to this repo?

@Roasbeef
Copy link
Member

^ the values should now be set 👍

@guggero guggero marked this pull request as ready for review July 13, 2022 07:15
@guggero guggero requested review from positiveblue and bhandras July 13, 2022 07:32
@guggero
Copy link
Contributor Author

guggero commented Jul 13, 2022

@@ -21,7 +21,7 @@ env:
# If you change this value, please change it in the following files as well:
# /.travis.yml
# /Dockerfile
GO_VERSION: 1.17.2
GO_VERSION: 1.17.12
Copy link
Member

Choose a reason for hiding this comment

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

not: why not 1.18?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are some unresolved problems with the linter and 1.18. We need to solve them in lnd first and then do the same in all other projects.

@guggero guggero merged commit f7c1ef2 into master Jul 15, 2022
@guggero guggero deleted the docker-build branch July 15, 2022 16:05
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.

4 participants