Skip to content

Conversation

@relaxolotl
Copy link
Contributor

APIs hidden behind features can now be viewed on docs.rs! All feature-gated modules and structs should be annotated with a note that mentions the specific feature flag needed to use the corresponding API.

This primarily fixes a linter warning regarding a link to a nonexistent integration which was gated behind a feature. There are other small fixes as well, mostly just hooking up links that weren't assigned destinations.

While the doc feature is considered unstable, we're taking advantage of the fact that docs are built with nightly by default.

APIs hidden behind features can now be viewed on docs.rs!
All feature-gated modules and structs should be annotated
with a note that mentions the specific feature flag needed
to use the corresponding API.

This primarily fixes a linter warning regarding a link to a
nonexistent integration which was gated behind a feature.
There are other small fixes as well, mostly just hooking up
links that weren't assigned destinations.

While the doc feature is considered unstable, we're taking
advantage of the fact that docs are built with nightly by
default.
@relaxolotl relaxolotl requested review from Swatinem and flub June 22, 2021 01:05
@codecov-commenter
Copy link

Codecov Report

Merging #346 (f5a1855) into master (0414b84) will increase coverage by 3.50%.
The diff coverage is 90.87%.

❗ Current head f5a1855 differs from pull request most recent head 6fb9534. Consider uploading reports for the commit 6fb9534 to get more accurate results

@@            Coverage Diff             @@
##           master     #346      +/-   ##
==========================================
+ Coverage   83.08%   86.58%   +3.50%     
==========================================
  Files          62       66       +4     
  Lines        6478     6838     +360     
==========================================
+ Hits         5382     5921     +539     
+ Misses       1096      917     -179     

@Swatinem Swatinem merged commit e2311e6 into master Jun 22, 2021
@Swatinem Swatinem deleted the docs/expose-featuregated-api branch June 22, 2021 07:47
Tuetuopay pushed a commit to Tuetuopay/sentry-rust that referenced this pull request Jul 5, 2021
APIs hidden behind features can now be viewed on docs.rs!
All feature-gated modules and structs should be annotated
with a note that mentions the specific feature flag needed
to use the corresponding API.

This primarily fixes a linter warning regarding a link to a
nonexistent integration which was gated behind a feature.
There are other small fixes as well, mostly just hooking up
links that weren't assigned destinations.

While the doc feature is considered unstable, we're taking
advantage of the fact that docs are built with nightly by
default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants