-
Notifications
You must be signed in to change notification settings - Fork 780
Privatise docsrs cfg flag; fix nightly
#2682
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
Conversation
cfg is a global namespace, and we don't want to trigger this in dependencies.
docsrs cfg flag; fix nightlydocsrs cfg flag; fix nightly
Benchmark resultsInstruction countsSignificant differencesClick to expand
Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2682 +/- ##
=======================================
Coverage 94.82% 94.82%
=======================================
Files 95 95
Lines 21819 21819
Branches 614 614
=======================================
Hits 20689 20689
Misses 1002 1002
Partials 128 128 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
djc
left a comment
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.
Makes sense. Should we backport all/part of this to rel-0.23?
The nightly feature has been removed in rust 1.92, see docs.rs build failure https://docs.rs/crate/rustls-native-certs/latest/builds/2773223 and rust PR rust-lang/rust#138907. Also, renamed the cfg into `rustls_native_certs_docsrs`. See rustls/rustls#2682 for more info.
cfg is a global namespace, and we don't want to trigger this in dependencies.
Related to rust-lang/rust#138907