Skip to content

Conversation

@ZeleOeO
Copy link
Contributor

@ZeleOeO ZeleOeO commented Jan 14, 2026

Resolves #1361

Using the established pattern in the workspace, this PR adds cargo-rdme markers for vello_hybrid. It ensures that the README.md is automatically synchronised with the module-level documentation in src/lib.rs

Just added cargo-rdme markers to vello_hybrid/README.md and then synchronized the files using cargo rdme --workspace-project=vello_hybrid.

You can verify by running cargo rdme --workspace-project=vello_hybrid --check. This will confirm the files are in sync without making changes.

Something to note is that the README did change. The descriptions and sections, features changed to key features and an architecture section was added.

Let me know if this is okay, thank you

cargo rdme --workspace-project=vello_hybrid --heading-base-level=0
Full documentation at https://github.com/orium/cargo-rdme -->

<<<<<<< HEAD
Copy link
Collaborator

Choose a reason for hiding this comment

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

Still has conflict markers that need to be resolved...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yep, can't believe i missed that. I'll fix that

@waywardmonkeys
Copy link
Collaborator

You'll also want to update .github/workflows/ci.yml to run the check during CI.

@ZeleOeO
Copy link
Contributor Author

ZeleOeO commented Jan 15, 2026

image

it's on the pipeline now

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.

Thanks. I've pushed a few changes myself:

  • I've removed the accidental reformatting of the rest of the ci.yml file.
  • I've removed the double header (pre-existing issue)
  • I've reduced the heading levels to match the usual expected Rust style (pre-existing issue)
  • I've removed some unused links (pre-existing issue)

In general, I recommend giving any PR you open a once-over in the GitHub "Files changed" tab, to see if anything looks amiss.

@DJMcNab DJMcNab enabled auto-merge January 16, 2026 10:13
@DJMcNab DJMcNab dismissed waywardmonkeys’s stale review January 16, 2026 10:46

The conflict markers are now gone

@DJMcNab DJMcNab added this pull request to the merge queue Jan 16, 2026
Merged via the queue into linebender:main with commit ac0b716 Jan 16, 2026
17 checks passed
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.

Add cargo-rdme to vello_hybrid to sync docs with README

3 participants