@b4n Thanks for having a look at the javascript parser. > we should probabmy use a newer snapshot (some parser changes, e.g. related to > class vs. object vs. variable are gonna lead to additional test result > changes, which are admittedly better in the newer version, and I'm not sure > it's worth having an intermediary step here)
Can do that. Should it be part of this PR or a separate one after this one is merged? > we should probably rather fix simple.js test to remove the invalid syntax > that confuses the parser. I'll try and do that Soon™. Could do it myself but better someone who at least remotely knows what the correct syntax is :-). > Tags p* are NOT releases, but probably pre-versions, or even nightly > snapshots. Releases are v*. It's probably fine, but noteworthy 🙂 OK, I didn't notice the release tags. But since these are just about once a year, I guess we want the latest thing and "risk" the consequences, right? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3859#issuecomment-2104519103 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/3859/c2104519...@github.com>