Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Bump parity-scale-codec to 3.6.1 #14428

Merged
merged 1 commit into from
Jun 21, 2023
Merged

Bump parity-scale-codec to 3.6.1 #14428

merged 1 commit into from
Jun 21, 2023

Conversation

koute
Copy link
Contributor

@koute koute commented Jun 21, 2023

This PR bumps parity-scale-codec to version 3.6.1.

In particular, this fixes the following two issues which can result in a stack overflow:

This was already merged in polkadot and cumulus; only substrate's left.

@koute koute added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jun 21, 2023
@koute koute requested review from a team June 21, 2023 10:33
@koute koute requested a review from a team June 21, 2023 10:33
@koute koute requested a review from cheme as a code owner June 21, 2023 10:33
@ggwpez
Copy link
Member

ggwpez commented Jun 21, 2023

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit e64926e into paritytech:master Jun 21, 2023
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
@koute @nathanwhit
Bump parity-scale-codec to 3.6.1 (paritytech#14428)
1568641
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ggwpez ggwpez ggwpez approved these changes

@davxy davxy davxy approved these changes

@cheme cheme cheme approved these changes

@athei athei Awaiting requested review from athei athei is a code owner

@acatangiu acatangiu Awaiting requested review from acatangiu acatangiu is a code owner

@sorpaas sorpaas Awaiting requested review from sorpaas sorpaas is a code owner

@andresilva andresilva Awaiting requested review from andresilva

@jsidorenko jsidorenko Awaiting requested review from jsidorenko

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
SDK Node
Status: done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@koute @ggwpez @cheme @davxy