Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Bump borsh from 0.10.3 to 0.11.0 (#31964)" #32145

Merged
merged 1 commit into from
Jun 15, 2023

Conversation

joncinque
Copy link
Contributor

This reverts commit 902d8b7.

Problem

According to near/borsh-rs#146, version 0.11 was yanked because there's a security advisory fix coming out, and 0.11 changes the serialization format. When people update to the version with the security fix, they would get some incompatibility.

Summary of Changes

Until that's resolved, let's just revert the upgrade.

Fixes #

Copy link
Contributor

@CriesofCarrots CriesofCarrots left a comment

Choose a reason for hiding this comment

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

Dang! Thanks for handling

@joncinque joncinque added the automerge Merge this Pull Request automatically once CI passes label Jun 14, 2023
@mergify mergify bot removed the automerge Merge this Pull Request automatically once CI passes label Jun 15, 2023
@mergify
Copy link
Contributor

mergify bot commented Jun 15, 2023

automerge label removed due to a CI failure

@joncinque joncinque added the automerge Merge this Pull Request automatically once CI passes label Jun 15, 2023
@mergify
Copy link
Contributor

mergify bot commented Jun 15, 2023

automerge label removed due to a CI failure

@mergify mergify bot removed the automerge Merge this Pull Request automatically once CI passes label Jun 15, 2023
@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Merging #32145 (b3df790) into master (4b30454) will decrease coverage by 0.1%.
The diff coverage is 11.1%.

@@            Coverage Diff            @@
##           master   #32145     +/-   ##
=========================================
- Coverage    81.9%    81.9%   -0.1%     
=========================================
  Files         767      767             
  Lines      208961   208980     +19     
=========================================
- Hits       171308   171297     -11     
- Misses      37653    37683     +30     

@joncinque joncinque merged commit bd4196b into solana-labs:master Jun 15, 2023
31 checks passed
@joncinque joncinque deleted the borsh-revert branch June 15, 2023 00:36
jeffwashington pushed a commit to HaoranYi/solana that referenced this pull request Jun 15, 2023
wen-coding pushed a commit to wen-coding/solana that referenced this pull request Aug 15, 2023
wen-coding pushed a commit to wen-coding/solana that referenced this pull request Aug 15, 2023
wen-coding pushed a commit to wen-coding/solana that referenced this pull request Aug 15, 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.

None yet

2 participants