Skip to content

Use cumulus package versions#809

Merged
alistair-singh merged 2 commits into
mainfrom
alistair/merge-from-upstream
Apr 18, 2023
Merged

Use cumulus package versions#809
alistair-singh merged 2 commits into
mainfrom
alistair/merge-from-upstream

Conversation

@alistair-singh

Copy link
Copy Markdown
Contributor
  • serde_json = "1.0.96"
  • serde = "1.0.160"
  • hex-literal = "0.4.1"

Cumulus Companion: Snowfork/cumulus#16

@vgeddes vgeddes left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

+1

@doubledup

Copy link
Copy Markdown
Contributor

Should we also bump scale-info to 2.5.0 in snowbridge-router-primitives and snowbridge-inbound-queue? Looks like it's updated in the Cumulus PR

@doubledup doubledup 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.

LGTM otherwise!

@alistair-singh

alistair-singh commented Apr 18, 2023

Copy link
Copy Markdown
Contributor Author

Should we also bump scale-info to 2.5.0 in snowbridge-router-primitives and snowbridge-inbound-queue? Looks like it's updated in the Cumulus PR

Nice catch! Added this in 595342e.

@alistair-singh alistair-singh force-pushed the alistair/merge-from-upstream branch from 595342e to bd604c5 Compare April 18, 2023 19:38
@alistair-singh alistair-singh merged commit 7d7d64a into main Apr 18, 2023
@alistair-singh alistair-singh deleted the alistair/merge-from-upstream branch April 18, 2023 20:32
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