Skip to content

QueryNode, metadata-protobuf and joystream/js package releases#4943

Merged
mnaamani merged 3 commits intoJoystream:masterfrom
mnaamani:master-qn-metadata-protobuf-release
Oct 24, 2023
Merged

QueryNode, metadata-protobuf and joystream/js package releases#4943
mnaamani merged 3 commits intoJoystream:masterfrom
mnaamani:master-qn-metadata-protobuf-release

Conversation

@mnaamani
Copy link
Copy Markdown
Member

Following #4927 preparing releases for affected packages.
Note bump to metadata-protobuf skipping v2.9.0 which was published from crt-release branch that included Token.proto which is being ported into this release on master branch.

Copy link
Copy Markdown
Contributor

@zeeshanakram3 zeeshanakram3 left a comment

Choose a reason for hiding this comment

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

The distributor-node/package.json & storage-node/package.json still have metadata-protobuf dependency with the previous version. Shouldn't we update the dependency at all places?

@mnaamani
Copy link
Copy Markdown
Member Author

The distributor-node/package.json & storage-node/package.json still have metadata-protobuf dependency with the previous version. Shouldn't we update the dependency at all places?

It was not strictly necessary for storage and distributor nodes because they do not actually use any of the new changes in the metadata-protobuf package, that is why I didn't update them.

Copy link
Copy Markdown
Contributor

@zeeshanakram3 zeeshanakram3 left a comment

Choose a reason for hiding this comment

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

LGTM

@mnaamani mnaamani merged commit 8013ffb into Joystream:master Oct 24, 2023
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.

2 participants