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

dev.Dockerfile: allow forcing a specific taproot-assets version #913

Conversation

ZZiigguurraatt
Copy link

dev.Dockerfile: allow forcing a specific taproot-assets version through a build argument.

An example of usage can be found here: https://github.com/lightninglabs/tapdvalidation/blob/cb7c752e9b123dc99882a471afa04cc1066e5111/docker-compose.yaml#L66

Copy link
Member

@guggero guggero left a comment

Choose a reason for hiding this comment

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

tACK, LGTM 🎉

dev.Dockerfile Outdated
@@ -23,10 +23,18 @@ COPY --from=nodejsbuilder /go/src/github.com/lightninglabs/lightning-terminal /g
# queries required to connect to linked containers succeed.
ENV GODEBUG netdns=cgo

# allow forcing a specific taproot-assets version through a build argument
Copy link
Member

Choose a reason for hiding this comment

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

nit: we use complete sentences including proper capitalization and punctuation for all comments in all our projects. And where possible (depending on the file format and content), we try to adhere to the 80 character limit outside of Go code as well.

Copy link
Author

Choose a reason for hiding this comment

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

fixed

@guggero guggero force-pushed the update-to-lnd-18-4 branch 4 times, most recently from dd49654 to 890764a Compare December 11, 2024 16:07
@ZZiigguurraatt ZZiigguurraatt force-pushed the taproot-assets-build-arg branch from 2db47f9 to e192d1e Compare December 11, 2024 17:49
@guggero guggero merged commit 1f4a0cd into lightninglabs:update-to-lnd-18-4 Dec 11, 2024
11 of 13 checks passed
@ZZiigguurraatt ZZiigguurraatt deleted the taproot-assets-build-arg branch December 11, 2024 18:33
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