David I. Lehn

Results 315 comments of David I. Lehn

The `.htaccess` needs to have that URL too. I proposed the changes.

It probably shouldn't leak memory. That code is rather ancient and likely working around issues in JS runtimes from 10-15 years ago. The techniques used there might be obsolete today.

Ah. I'm not sure how that code works anymore. If we modernize all the code, will have to look at dealing with these sorts of issues. If you have insights...

Also note we will (hopefully soon) update to eslint v9 and I had thought of using their new https://github.com/eslint/markdown plugin. That might not be as comprehensive yet though.

What's a good way to add a test for this?

- Related: https://github.com/digitalbazaar/jsonld.js/pull/555. - The ESM/CJS style mixing can get tricky sometimes. I was hoping to avoid mentioning it at all and move to ESM where we can control it...

- The code style is different. Extra newlines and such. Would be better to match what is there. - None of the other code has type annotations. (This code pre-dates...

> Conversion of this document to a RDF dataset works at JSON-LD playground, but it will crash pyld 2.0.4: Please don't assume "it works in the json-ld.org playground" means the...

Thanks for the analysis. I think we're in the same thinking as many other projects and people that we are not quite sure how to handle LLM generated code at...

I assume you want [luizbonino](https://github.com/luizbonino) to approve?