todo

[tbo47]

• remove * in imports
• write jsdoc to improve the types generation. With all the feedback from Typescript migration #28 tweak typescript integration #30 typescript-eslint #29
• is there more unit tests to import from legacy project?

[tbo47]

@aloisklink thoughts?

[aloisklink]

I 100% agree with all of them!

• write jsdoc to improve the types generation. With all the feedback from Typescript migration #28 tweak typescript integration #30 typescript-eslint #29

👍

I think if we add a /** @param {Graph} g */ to most functions that use g as a parameter, that would greatly improve the types, and should hopefully catch any future bugs if we do try to do any lodash migrations/refactoring in the future!

• is there more unit tests to import from legacy project?

I don't think there is.

There are some browser tests in the dagre-d3 repo that we could use: https://github.com/dagrejs/dagre-d3/tree/master/test, but because they need to run in a browser, adding them to this repo would be much harder. The test library they're using is pretty outdated, so it'll be quite a lot of work to use here.

But yes! I'd love to at least have one unit test that could have detected #40 (comment)