Skip to content

Geometric tilt readme fix#656

Merged
marvin-hansen merged 2 commits into
deepcausality-rs:mainfrom
RowanTL:geometric_tilt_readme_fix
Jun 28, 2026
Merged

Geometric tilt readme fix#656
marvin-hansen merged 2 commits into
deepcausality-rs:mainfrom
RowanTL:geometric_tilt_readme_fix

Conversation

@RowanTL

@RowanTL RowanTL commented Jun 28, 2026

Copy link
Copy Markdown

Describe your changes

The cargo run command inside of the geometric_tilt example was incorrect. This PR fixes that line.

Issue ticket number and link

Code checklist before requesting a review

  • I have signed the DCO?
  • All tests are passing when running make test?
  • No errors or security vulnerabilities are reported by make check?

For details on make, please see BUILD.md

Note: The CI runs all of the above and fixing things before they hit CI speeds
up the review and merge process. Thank you.


Summary by cubic

Fixes the run command in the geometric_tilt example README so the example runs correctly. Also adds RowanTL to MAINTAINERS.md.

  • Bug Fixes
    • Updated README command to: cargo run -p physics_examples --example geometric_tilt

Written for commit 3adda72. Summary will update on new commits.

Review in cubic

Rowan Torbitzky-Lane added 2 commits June 28, 2026 08:55
…ion of readme

Signed-off-by: Rowan Torbitzky-Lane <rowan.a.tl@protonmail.com>
Signed-off-by: Rowan Torbitzky-Lane <rowan.a.tl@protonmail.com>
@RowanTL RowanTL requested a review from marvin-hansen as a code owner June 28, 2026 16:02

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 2 files

Re-trigger cubic

@marvin-hansen marvin-hansen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@marvin-hansen marvin-hansen merged commit 4b17606 into deepcausality-rs:main Jun 28, 2026
15 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.

2 participants