Skip to content

Updating upstream dependencies#4345

Merged
alexanderbez merged 2 commits into
cosmos:masterfrom
Zondax:nanox
May 16, 2019
Merged

Updating upstream dependencies#4345
alexanderbez merged 2 commits into
cosmos:masterfrom
Zondax:nanox

Conversation

@jleni

@jleni jleni commented May 16, 2019

Copy link
Copy Markdown
Contributor

There have been reports of Nano X detection issues in gaiacli.
This PR fixes the problem by upgrading upstream dependencies.

Related PRs:
cosmos/ledger-cosmos-go#8
cosmos/ledger-go#2

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
  • Wrote tests
  • Updated relevant documentation (docs/)
  • Added a relevant changelog entry: clog add [section] [stanza] [message]
  • rereviewed Files changed in the github PR explorer

For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@jleni jleni marked this pull request as ready for review May 16, 2019 14:48

@alexanderbez alexanderbez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ACK

@alexanderbez alexanderbez merged commit e09ebf9 into cosmos:master May 16, 2019
@alexanderbez alexanderbez mentioned this pull request May 23, 2019
5 tasks
alexanderbez added a commit that referenced this pull request May 23, 2019
* Cherry Pick PR #4345: Upgrade ledger-cosmos-go

* Cherry Pick PR #4336: Fix AppendTags usage error

* Update modules

* Cherry Pick PR #4265: CacheKVStore keep sorted items

* Cherry Pick #4227: Adding support for Ledger Cosmos App v1.5

* Cherry Pick #4395: Improve sig verification error message

* Cherry Pick PR #4140: Fix Failed Simulation Seeds
@rllola rllola deleted the nanox branch November 21, 2022 13:29
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