Skip to content

deny(missing_docs) for bevy_mikktspace #19654

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

Merged
merged 3 commits into from
Jun 15, 2025

Conversation

theotherphil
Copy link
Contributor

@theotherphil theotherphil commented Jun 15, 2025

Objective

Remove allow(missing_docs) from bevy_mikktspace and add a simple top-level doc comment, towards #3492

@theotherphil theotherphil changed the title [EXPERIMENT] Remove allow(missing_docs) from bevy_mikktspace [DRAFT] deny(missing_docs) for bevy_mikktspace Jun 15, 2025
@theotherphil theotherphil added C-Docs An addition or correction to our documentation D-Trivial Nice and easy! A great choice to get started with Bevy labels Jun 15, 2025
@theotherphil theotherphil marked this pull request as ready for review June 15, 2025 14:10
@theotherphil theotherphil changed the title [DRAFT] deny(missing_docs) for bevy_mikktspace deny(missing_docs) for bevy_mikktspace Jun 15, 2025
@theotherphil theotherphil added the S-Needs-Review Needs reviewer attention (from anyone!) to move forward label Jun 15, 2025
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jun 15, 2025
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jun 15, 2025
Merged via the queue into bevyengine:main with commit 510efdf Jun 15, 2025
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Docs An addition or correction to our documentation D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants