-
Notifications
You must be signed in to change notification settings - Fork 612
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
Use cargo-check-external-types to control type leakage in public API #1535
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1535 +/- ##
==========================================
- Coverage 96.46% 96.46% -0.01%
==========================================
Files 74 74
Lines 15115 15107 -8
==========================================
- Hits 14581 14573 -8
Misses 534 534
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
Nice! This is super handy 🚀
Working with upstream to get this going with a more recent nightly. |
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.
Looks like this is working nicely with up-to-date nightly thanks to your upstream contrib. Cool!
This needs nightly, which is affixed as the version documneted as working by cargo-check-external-types. external-types.toml is a config file as a starting point: it allows all types from pki-types. This currently fails due to some `impl From<ExternalType>` on public types.
0fd42e3
to
5fe5272
Compare
I learned about this today, so thought I'd give it a try. It pointed out one
rustls-webpki
and tworing
leakages that this PR fixes.See https://github.com/rustls/rustls/actions/runs/6511651812/job/17687753330 for the example failing output.