Sebastian Schuberth

Results 1446 comments of Sebastian Schuberth

> Consider adding information about the breaking change to the commit message and how users can deal with it. I've reworded the commit message. Can we now please move forward...

> Should this paragraph be changed too: Done in a separate commit.

For the record, it seems this regression did not surface before https://github.com/oss-review-toolkit/ort/pull/9616, although it was introduced before that.

> the analysis produces results up until 39.0.0, but starting with 40.0.0 it aborts Thank for this analysis! > Sadly I am not yet familiar enough with the code base...

> For `npm` IIRC there has never been support for workspaces implemented, nor there were tests for it. That's likely true, though analysis at least *seemed* to work before, not...

> We need ORT with NPM project support! @thilo20, have you considered sponsoring the project then to be able to prioritize this issue?

> Would the analyzer result contain `git://` URLs then? I believe so, yes.

For testing, also see https://github.com/wix/Detox/issues/4871, maybe.

> For testing, also see [wix/Detox#4871](https://github.com/wix/Detox/issues/4871), maybe. For the record, I just verified that plain `git` CLI is able to recursively clone that repo when configuring `git config --global url."https://github.com/".insteadOf...