Chore: Bump lezer #8
Conversation
|
|
There was a problem hiding this comment.
Actually, I've found out that tree viz is not working with this change. +cc @gabor
| "@lezer/lr": "https://unpkg.com/@lezer/lr@0.15.8/dist/index.js", | ||
| "@lezer/common": "https://unpkg.com/@lezer/common@0.15.11/dist/index.js" | ||
| "@grafana/lezer-logql": "https://unpkg.com/@grafana/lezer-logql@0.0.13/index.es.js", | ||
| "@lezer/lr": "https://unpkg.com/@lezer/lr@1.0.0/dist/index.js", |
There was a problem hiding this comment.
Actually, this @lezer/lr": "https://unpkg.com/@lezer/lr@1.0.0/dist/index.js specific change breaks the tree viz tool. It gives module import failed. this requires a browser with import map support.. The issue is present from @lezer/lr@16.0.0 and 15.8 is the last working version for this.
There was a problem hiding this comment.
I think it shouldn't until we release this.
There was a problem hiding this comment.
@gabor not sure if for the browser tool it make sense to have different version for now and create issue in their repo?
|
@ivanahuckova hmm.. i'll look into this problem, but i think for now we can merge this, we can fix the |
Please take care of releasing this if you find it useful.