Conversation
No issues found. The latest commit generalizes hardcoded Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
There was a problem hiding this comment.
Pull request overview
This PR enables Ledger support by building the chain with the ledger build tag and updates protobuf dependency locks accordingly.
Changes:
- Add Makefile support for passing
--build.tags "ledger"duringignite chain build. - Update GitHub Actions workflows (system tests + release) to build with the
ledgertag. - Update
buf.lockdependency commits/digests (notablycosmos-sdkandwellknowntypes).
Reviewed changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
buf.lock |
Updates locked buf dependencies to newer commits/digests. |
Makefile |
Introduces build-tag handling and applies it to build/release commands. |
.github/workflows/systemtests.yaml |
Builds the chain with --build.tags "ledger" in CI system tests. |
.github/workflows/release.yml |
Builds release artifacts with --build.tags "ledger". |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ith BUILD_TAGS usage
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.