Skip to content
This repository was archived by the owner on Jan 26, 2023. It is now read-only.

Conversation

@hansl
Copy link
Contributor

@hansl hansl commented Oct 6, 2022

This will make sure we get at least the SHA of the commit that was used for production.

@hansl hansl requested a review from fmorency October 6, 2022 00:44
@hansl hansl self-assigned this Oct 6, 2022
Copy link
Contributor

@fmorency fmorency left a comment

Choose a reason for hiding this comment

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

LGTM assuming the comment is addressed. Thanks!

tendermint-rpc = { version = "0.24.0-pre.2", features = [ "http-client" ] }
tendermint-proto = "0.24.0-pre.2"
tokio = { version = "1.13.0", features = [ "full" ] }
tracing = "0.1.28"
Copy link
Contributor

Choose a reason for hiding this comment

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

According to the vergen documentation, you are missing build = "build.rs" in the [package] section of this file.

That being said, it seems to work anyway.

@fmorency fmorency added the enhancement New feature or request label Oct 6, 2022
@hansl hansl force-pushed the add-vergen-to-abci branch from aacdce4 to 05d6466 Compare October 6, 2022 17:02
@hansl hansl merged commit 2d7addf into liftedinit:main Oct 6, 2022
@hansl hansl deleted the add-vergen-to-abci branch October 7, 2022 02:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants