Sebastian Schuberth

Results 1514 comments of Sebastian Schuberth

Just misc. comments / some ideas: > Caching of analyzer results Maybe we could use the `UploadResultToPostgresCommand` as a basis for storing analyzer results, and then reusing them. > It...

> > Also @oheger-bosch was briefly looking at Exposed for using a data source / connection pooling, but came across [JetBrains/Exposed#127](https://github.com/JetBrains/Exposed/issues/127). How does Exposed work for you without JSON support?...

> I'm fine with a central server (definitely if it simplifies implementation). Also see https://github.com/oss-review-toolkit/ort/issues/4688.

> Possible improvements on the infrastructure side of things These topics by now better fit to https://github.com/eclipse-apoapsis/ort-server. As other topics have been mostly addressed, I'm closing this issue after the...

> Fixes #6235. I did a quick search across the code base, and I've found a use of `toSortedSet()` (thought not in the model) that you might want to remove...

Test failures are related to outdated Pub tests only, see https://github.com/oss-review-toolkit/ort/pull/8722.

Maybe a way forward could be to leverage [nvm-rust](https://github.com/BeeeQueue/nvm-rust) to install the correct Node version on-the-fly, and then use [Corepack](https://nodejs.org/api/corepack.html) to install the right Node package manager version.

Closed as part of backlog grooming. Feel free to comment if you would like to contribute to this.

Agreed with @oss-review-toolkit/core-devs that we always want to simplify `AND` expressions like this; maybe some post-processing of license choices is required here.

@stevespringett mind having a look?