Skip to content

[Merged by Bors] - Link Minimal and Default plugins in the docs.#2583

Closed
mirkoRainer wants to merge 2 commits intobevyengine:mainfrom
mirkoRainer:mirko/default_plugin_doc_update
Closed

[Merged by Bors] - Link Minimal and Default plugins in the docs.#2583
mirkoRainer wants to merge 2 commits intobevyengine:mainfrom
mirkoRainer:mirko/default_plugin_doc_update

Conversation

@mirkoRainer
Copy link
Contributor

I didn't know about MinimalPlugins for way too long. This should increase visibility for others.

Objective

Improve visibility and discover in the docs for Default and Minimal Plugins.

Solution

Links the two Docs pages.

I didn't know about MinimalPlugins for way too long. This should increase visibility for folk.
@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Aug 2, 2021
@Nilirad
Copy link
Contributor

Nilirad commented Aug 2, 2021

I don't think the emphasis on the text is necessary. Also it would be good to explain the advantages of the other solution (e.g. See also DefaultPlugins for a more complete solution).

Adjusting wording. Removing emphasis.
@mirkoRainer
Copy link
Contributor Author

I don't think the emphasis on the text is necessary. Also it would be good to explain the advantages of the other solution (e.g. See also DefaultPlugins for a more complete solution).

Adjusted.

@NiklasEi NiklasEi added C-Docs An addition or correction to our documentation and removed S-Needs-Triage This issue needs to be labelled labels Aug 6, 2021
@mockersf mockersf added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Aug 9, 2021
@mockersf
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Aug 10, 2021
I didn't know about MinimalPlugins for way too long. This should increase visibility for others.

# Objective

Improve visibility and discover in the docs for Default and Minimal Plugins.

## Solution

Links the two Docs pages. 



Co-authored-by: Mirko Rainer <52899592+mirkoRainer@users.noreply.github.com>
@bors bors bot changed the title Link Minimal and Default plugins in the docs. [Merged by Bors] - Link Minimal and Default plugins in the docs. Aug 11, 2021
@bors bors bot closed this Aug 11, 2021
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 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.

4 participants