Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Update crossbeam-deque to 0.8.1 #19361

Merged
merged 1 commit into from
Aug 23, 2021
Merged

Conversation

brooksprumo
Copy link
Contributor

@brooksprumo brooksprumo commented Aug 22, 2021

Problem

Builds were broken due to a vulnerability in crossbeam-deque.

Summary of Changes

Update crossbeam-deque.

@brooksprumo brooksprumo marked this pull request as ready for review August 23, 2021 01:51
Copy link
Contributor

@t-nelson t-nelson left a comment

Choose a reason for hiding this comment

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

lgtm!

@t-nelson t-nelson added automerge Merge this Pull Request automatically once CI passes v1.6 labels Aug 23, 2021
@mergify mergify bot removed the automerge Merge this Pull Request automatically once CI passes label Aug 23, 2021
@mergify
Copy link
Contributor

mergify bot commented Aug 23, 2021

automerge label removed due to a CI failure

@brooksprumo brooksprumo added the automerge Merge this Pull Request automatically once CI passes label Aug 23, 2021
@codecov
Copy link

codecov bot commented Aug 23, 2021

Codecov Report

Merging #19361 (e3a07d5) into master (6d93981) will decrease coverage by 0.0%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #19361     +/-   ##
=========================================
- Coverage    82.8%    82.8%   -0.1%     
=========================================
  Files         457      457             
  Lines      130682   130682             
=========================================
- Hits       108289   108257     -32     
- Misses      22393    22425     +32     

@mergify mergify bot merged commit 2ccbe47 into solana-labs:master Aug 23, 2021
mergify bot pushed a commit that referenced this pull request Aug 23, 2021
(cherry picked from commit 2ccbe47)

# Conflicts:
#	Cargo.lock
mergify bot pushed a commit that referenced this pull request Aug 23, 2021
(cherry picked from commit 2ccbe47)

# Conflicts:
#	Cargo.lock
@CriesofCarrots
Copy link
Contributor

Thanks, @brooksprumo ! Sorry I missed the review.
I may go ahead and bump crossbeam-epoch on master to get rid of the warning... unless there was an explicit reason you decided not to?

@brooksprumo
Copy link
Contributor Author

Thanks, @brooksprumo ! Sorry I missed the review.

I may go ahead and bump crossbeam-epoch on master to get rid of the warning... unless there was an explicit reason you decided not to?

@CriesofCarrots No worries! I didn't bump crossbeam-epoch also because I just wanted to do the smallest change necessary.

brooksprumo added a commit to brooksprumo/solana that referenced this pull request Aug 23, 2021
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
brooksprumo added a commit to brooksprumo/solana that referenced this pull request Aug 23, 2021
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
mergify bot pushed a commit that referenced this pull request Aug 23, 2021
mergify bot pushed a commit that referenced this pull request Aug 24, 2021
@brooksprumo brooksprumo deleted the deps branch September 20, 2021 21:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants