Skip to content

feat: Add support for laying out graphs horizontally - #301

Open
mlyoung101 wants to merge 1 commit into
bobluppes:mainfrom
Precision-Contribution-Group:io-add-layout-direction
Open

feat: Add support for laying out graphs horizontally#301
mlyoung101 wants to merge 1 commit into
bobluppes:mainfrom
Precision-Contribution-Group:io-add-layout-direction

Conversation

@mlyoung101

Copy link
Copy Markdown
Contributor

For my project, the graphs work much better when laid out horizontally, so I added this as a feature upstream. It just adds a flag to io::to_dot called horizontal, that defaults to false to preserve existing behaviour. Tests added too :)

@mlyoung101
mlyoung101 force-pushed the io-add-layout-direction branch from 51b86ae to b82914d Compare January 11, 2026 08:58
@codecov

codecov Bot commented Jan 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.58%. Comparing base (8b0701d) to head (b82914d).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #301   +/-   ##
=======================================
  Coverage   99.58%   99.58%           
=======================================
  Files          59       59           
  Lines        2621     2637   +16     
  Branches      136      137    +1     
=======================================
+ Hits         2610     2626   +16     
  Misses         11       11           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

Copy link
Copy Markdown
Contributor

Marking this PR as stale. It will not be automatically closed.

Even though the maintainers of Graaf may not always have time to take a look in a timely fashion, your contributions are much appreciated.
Please allow some time for @bobluppes to take a closer look.

@github-actions github-actions Bot added the stale label Feb 10, 2026
@mlyoung101

Copy link
Copy Markdown
Contributor Author

Hey @bobluppes, if you have the chance do you reckon you could have a look at this? It's a pretty short diff.

@github-actions github-actions Bot removed the stale label Feb 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Marking this PR as stale. It will not be automatically closed.

Even though the maintainers of Graaf may not always have time to take a look in a timely fashion, your contributions are much appreciated.
Please allow some time for @bobluppes to take a closer look.

@github-actions github-actions Bot added the stale label Mar 14, 2026
@github-actions github-actions Bot removed the stale label Jul 22, 2026
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.

1 participant