Skip to content

Conversation

@jonasbb
Copy link
Owner

@jonasbb jonasbb commented Aug 18, 2023

Document the feature flags in the Cargo.toml file instead of a separate
file. Keep the old file, but generate the content from the Cargo.toml
using the document-features crate.

The benefit of this approach is that the documentation is stored next to
the features. lib.rs has a new feature that reads the documentation from
there and shows it as part of the feature list.

bors r+

Document the feature flags in the Cargo.toml file instead of a separate
file. Keep the old file, but generate the content from the Cargo.toml
using the `document-features` crate.

The benefit of this approach is that the documentation is stored next to
the features. lib.rs has a new feature that reads the documentation from
there and shows it as part of the feature list.
@codecov
Copy link

codecov bot commented Aug 18, 2023

Codecov Report

Merging #639 (76753d8) into master (eb48aee) will not change coverage.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #639   +/-   ##
=======================================
  Coverage   64.68%   64.68%           
=======================================
  Files          36       36           
  Lines        2186     2186           
=======================================
  Hits         1414     1414           
  Misses        772      772           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bors
Copy link
Contributor

bors bot commented Aug 18, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 5687d0f into master Aug 18, 2023
@bors bors bot deleted the better-feature-documentation branch August 18, 2023 14:59
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.

2 participants