-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Conversation
Codecov Report
@@ 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
serde = { version = "1.0.112", features = ["derive"] } | ||
serde_json = "1.0.56" |
There was a problem hiding this comment.
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
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 😅 |
can't remember who is the first one use this but it is really useful 😈 |
Problem
Some crates don't use workspace inheritance
Summary of Changes
make them use it ⛏️