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

fix: use cli feature for cargo chainhook-install #486

Merged
merged 1 commit into from Feb 6, 2024

Conversation

MicaiahReid
Copy link
Collaborator

Previously, the cargo chainhook-install script was installing the sdk version of Chainhook, which logged data in a non-user-friendly way. This changes our features some and updates the install script to fix those logs.

Copy link
Member

@lgalabru lgalabru left a comment

Choose a reason for hiding this comment

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

LGTM

@MicaiahReid MicaiahReid merged commit 1588b8f into develop Feb 6, 2024
3 checks passed
@MicaiahReid MicaiahReid deleted the fix-cli-output branch February 6, 2024 15:12
MicaiahReid added a commit that referenced this pull request Feb 8, 2024
Previously, the `cargo chainhook-install` script was installing the sdk
version of Chainhook, which logged data in a non-user-friendly way. This
changes our features some and updates the install script to fix those
logs.
github-actions bot pushed a commit that referenced this pull request Feb 8, 2024
## [1.3.0](v1.2.1...v1.3.0) (2024-02-08)

### Features

* optionally serve Prometheus metrics ([#473](#473)) ([67a38ac](67a38ac))

### Bug Fixes

* adjust ordinal_number entry in ts client inscription transfer event, add new reveal data ([#476](#476)) ([28bf5c4](28bf5c4))
* remove early return for event evaluation ([#484](#484)) ([98f9e86](98f9e86)), closes [#469](#469)
* remove unreachable panic; return instead ([#490](#490)) ([abe0fd5](abe0fd5))
* use cli feature for `cargo chainhook-install` ([#486](#486)) ([32f4d4e](32f4d4e))
* validate predicate `start_block` and `end_block` ([#489](#489)) ([e70025b](e70025b)), closes [#477](#477) [#464](#464)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants