search

comunica / sparqlee

⚙️ SPARQL expression evaluator library - Moved to @comunica/expression-evaluator
https://www.npmjs.com/package/sparqlee
14 stars 7 forks source link

Removes BlankNode.nextID #148

Closed Tpt closed 2 years ago

Tpt commented 2 years ago

Unused

Tpt commented 2 years ago

It's a breaking change so I am not sure it is welcome. If it is better, I can easily cover this function with a test.

coveralls commented 2 years ago

Coverage Status

Coverage increased (+0.08%) to 97.448% when pulling 9f5209974d7bca10c87a8d3813029db89fdd780e on Tpt:bnode-next-id into 969819abfee158837189d62aa5b8a57ca8bfc5a6 on comunica:master.

rubensworks commented 2 years ago

AFAICS, it's not being used in Comunica either. So it's fine to remove this.