-
Notifications
You must be signed in to change notification settings - Fork 126
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
Use cargo-rdme
to sync README and documentation root
#435
Conversation
Not sure what's up with the "expected check". I guess this is because I renamed one of the CI tasks, but I don't know if that will block a merge or what the proper way to do a rename is. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Once we settle on the new name, I can fixup the branch protection
I do want to reorder some of the items slightly, but we can have this discussion on Zulip afterwards if needed.
Some screenshots to feed the discussion: Masonry README before the changeMasonry README after the changeDoc page with the blurb centeredDoc page with badges and the blurb, both centeredOverall my takeaways are:
Will push follow-up commit soon based on those takeaways. |
New version pushed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're still blocking on the name for the CI check. If you want to leave it as-is, please indicate as such, and we can land this.
@@ -1,15 +1,18 @@ | |||
// Copyright 2018 the Xilem Authors and the Druid Authors | |||
// SPDX-License-Identifier: Apache-2.0 | |||
|
|||
//! A framework that aims to provide the foundation for Rust GUI libraries. | |||
//! | |||
//! Masonry gives you a platform to create windows (using [winit] as a backend) each with a tree of widgets. It also gives you tools to inspect that widget tree at runtime, write unit tests on it, and generally have an easier time debugging and maintaining your app. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this first sentence should be rewritten, but that's not required in this PR.
Co-authored-by: Daniel McNab <36049421+DJMcNab@users.noreply.github.com>
Changed the name as per your commit. I'd say this is good to merge. |
See linebender/xilem#435 and https://github.com/orium/cargo-rdme Also adds documentation for the `std` and `libm` features
See linebender/xilem#435 and https://github.com/orium/cargo-rdme Also adds documentation for the `std` and `libm` features
This was remarkably painless.
I especially appreciate @taiki-e providing an install-action which does 90% of the CI work for us.