Juho Lauri

As a side question, why does IGraphM have `IGCompleteQ`? Does it somehow differ from the one already in Mathematica, or which one should I use?

Technically, either (block or cactus) can be disconnected. But maybe it's ok to require connectedness in both cases (but the docs should say this) - the user can just run...

On a closer thought, I think it indeed makes sense to require connectnedness here. The reason is that these graphs are often defined via a block-cut tree which is a...

> @jplauri regarding your question about merge vs insert, i think merge is the right call. That list of issues is pretty ancient now. Merge was implemented in 2008 and...

@dstandish Would you be able to advise me further here, i.e., is there something more the PR should have? Should we just wait for a review/merge, or what's the process?...

@dstandish Not really sure if I got it right - I basically copied from TestPostgres with only superficial understanding. But intuitively it makes sense - running `test_mssql_to_mssql` will fail unless...

> Please add a test for this Should this be good to go now?

Just to avoid this going stale as I think it could be helpful. @dstandish do you think some work is still required?

> Please add a test for this Would you have a chance to revisit this PR? Thanks!