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

chore: workspace inheritance #30939

Merged
merged 16 commits into from
Mar 29, 2023
Merged

Conversation

yihau
Copy link
Contributor

@yihau yihau commented Mar 28, 2023

Problem

Some crates don't use workspace inheritance

Summary of Changes

make them use it ⛏️

@yihau yihau changed the title Use cargo inheritance chore: cargo inheritance Mar 28, 2023
@codecov
Copy link

codecov bot commented Mar 28, 2023

Codecov Report

Merging #30939 (8db8e82) into master (77aac98) will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #30939     +/-   ##
=========================================
- Coverage    81.5%    81.5%   -0.1%     
=========================================
  Files         727      727             
  Lines      205250   205250             
=========================================
- Hits       167430   167424      -6     
- Misses      37820    37826      +6     

@yihau yihau marked this pull request as ready for review March 29, 2023 03:18
@yihau yihau changed the title chore: cargo inheritance chore: workspace inheritance Mar 29, 2023
@yihau yihau added the automerge Merge this Pull Request automatically once CI passes label Mar 29, 2023
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!

Comment on lines -14 to -15
serde = { version = "1.0.112", features = ["derive"] }
serde_json = "1.0.56"
Copy link
Contributor

Choose a reason for hiding this comment

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

both of these will be updated by moving to the workspace version. probably not a problem, just wanted to raise

@mergify mergify bot merged commit 2c842e9 into solana-labs:master Mar 29, 2023
@CriesofCarrots
Copy link
Contributor

Whoa, TIL you can set automerge before you actually receive review approval.

@t-nelson
Copy link
Contributor

Whoa, TIL you can set automerge before you actually receive review approval.

same! got scared for a second that i'd merged it by accident 😅

@yihau
Copy link
Contributor Author

yihau commented Mar 29, 2023

can't remember who is the first one use this but it is really useful 😈

@yihau yihau deleted the use-cargo-inheritance branch March 29, 2023 03:53
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