Skip to content
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

Merged
merged 3 commits into from
Jul 22, 2024

Conversation

PoignardAzur
Copy link
Contributor

This was remarkably painless.

I especially appreciate @taiki-e providing an install-action which does 90% of the CI work for us.

@PoignardAzur PoignardAzur requested a review from DJMcNab July 18, 2024 10:33
@PoignardAzur
Copy link
Contributor Author

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.

Copy link
Member

@DJMcNab DJMcNab left a 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.

masonry/README.md Outdated Show resolved Hide resolved
masonry/README.md Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@PoignardAzur PoignardAzur enabled auto-merge July 18, 2024 15:05
@PoignardAzur
Copy link
Contributor Author

PoignardAzur commented Jul 21, 2024

Some screenshots to feed the discussion:

Masonry README before the change

image

Masonry README after the change

image

Doc page with the blurb centered

image

Doc page with badges and the blurb, both centered

image


Overall my takeaways are:

  • Anything centered looks bad in the generated doc.
  • In the README, having badges below the title and blurb does look better.
  • I don't think the doc really needs the blurb.

Will push follow-up commit soon based on those takeaways.

@PoignardAzur
Copy link
Contributor Author

New version pushed.

RENDERED.

Copy link
Member

@DJMcNab DJMcNab left a 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.
Copy link
Member

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>
@PoignardAzur
Copy link
Contributor Author

We're still blocking on the name for the CI check

Changed the name as per your commit. I'd say this is good to merge.

@PoignardAzur PoignardAzur added this pull request to the merge queue Jul 22, 2024
Merged via the queue into main with commit fc6a522 Jul 22, 2024
16 checks passed
@PoignardAzur PoignardAzur deleted the use-cargo-rdme branch July 22, 2024 13:01
DJMcNab added a commit to DJMcNab/color that referenced this pull request Nov 13, 2024
See linebender/xilem#435
and https://github.com/orium/cargo-rdme

Also adds documentation for the `std` and `libm` features
github-merge-queue bot pushed a commit to linebender/color that referenced this pull request Nov 13, 2024
See linebender/xilem#435 and
https://github.com/orium/cargo-rdme

Also adds documentation for the `std` and `libm` features
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