Skip to content

Remove old docs that no longer apply#24838

Open
ItsDoot wants to merge 1 commit into
bevyengine:mainfrom
ItsDoot:docs/winitplugin
Open

Remove old docs that no longer apply#24838
ItsDoot wants to merge 1 commit into
bevyengine:mainfrom
ItsDoot:docs/winitplugin

Conversation

@ItsDoot

@ItsDoot ItsDoot commented Jul 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Objective

WinitPlugin's docs mention some M generic, which is no longer present on the struct as of #22088:

Made WinitPlugin no longer generic

Solution

Remove them.

@ItsDoot ItsDoot added C-Docs An addition or correction to our documentation D-Trivial Nice and easy! A great choice to get started with Bevy A-Windowing Platform-agnostic interface layer to run your app in D-Domain-Agnostic Can be tackled by anyone with generic programming or Rust skills S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jul 2, 2026
eswartz
eswartz approved these changes Jul 2, 2026
View reviewed changes

@eswartz eswartz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@Zeophlite Zeophlite 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 Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@eswartz eswartz eswartz approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A-Windowing Platform-agnostic interface layer to run your app in C-Docs An addition or correction to our documentation D-Domain-Agnostic Can be tackled by anyone with generic programming or Rust skills 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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ItsDoot @eswartz @Zeophlite