Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Jan 30, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

aloisklink and others added 3 commits January 29, 2026 20:13
The order of the arguments for `graph.setEdge()` in
`createGraphWithElements` is incorrect.

I've updated the call to instead use an object to make it a bit more
clear, since the order is a bit unusual.

This function isn't actually used anywhere in our code, which is why
this is only a `refactor:` commit, not a `fix:` commit.
But from talking to @knsv, it sounds like this is going to be used by
future layout algorithms, so we should keep this file.

But, this  unblocks upgrading to `dagre-d3-es` `7.0.14`,
which has updated types for this.
This release includes some basic TypeScript types, which makes our life
easier!

See: https://github.com/tbo47/dagre-es/releases/tag/7.0.14
…aphWithElements-function

refactor: update `dagre-d3-es` to v7.0.14
@pull pull bot locked and limited conversation to collaborators Jan 30, 2026
@pull pull bot added the ⤵️ pull label Jan 30, 2026
@pull pull bot merged commit d4611b5 into code:develop Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants