-
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
Fix docs.rs build #1827
Fix docs.rs build #1827
Conversation
The docs.rs environment has golang installed, but doesn't have the environment variables needed to make it actually work: rust-lang/docs.rs#1303 So avoid that entirely.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1827 +/- ##
=======================================
Coverage 95.83% 95.83%
=======================================
Files 84 84
Lines 18861 18879 +18
=======================================
+ Hits 18075 18093 +18
Misses 786 786 ☔ View full report in Codecov by Sentry. |
Benchmark resultsInstruction countsSignificant differencesThere are no significant instruction count differences Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
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.
Thanks for fixing 👍
See rust-lang/docs.rs#1303: docs.rs's build environment has a golang package but doesn't set the environment variables needed to make it usable.
This PR avoids that by not generating docs with the
fips
feature, and then directly addressing the one API item we have under thefips
feature.fixes #1826