ObserverOfTime comments

Results 325 comments of


                                            ObserverOfTime

ci: update main workflow

`tree-sitter/parser-setup-action` is replaced by `actions/checkout` + `tree-sitter/setup-action`. `tree-sitter/parser-test-action@v2` no longer calls `tree-sitter/parse-action` automatically.

no executable found for arm64 devices on releases later than v0.23.1

Use tree-sitter 0.25 and tree-sitter-bash 0.25

no executable found for arm64 devices on releases later than v0.23.1

Oh yeah, that's tree-sitter/node-tree-sitter#248

tree-sitter/highlight example doesn't work

Does it work if you use the latest versions?

tree-sitter/highlight example doesn't work

Then would you mind submitting a PR to update the versions in the file?

Use tabs only in recipes so as to avoid confusing 'make'

Windows is not supported by the Makefile. The issue here is that it fails without printing the actual error.

Use tabs only in recipes so as to avoid confusing 'make'

> I found out that building tree-sitter 0.22.6 works for us on Windows It works in shells that emulate Unix and builds Unix-like libraries. It does not (normally) work in...

Use tabs only in recipes so as to avoid confusing 'make'

> All we need is a Windows check that excludes Cygwin. What about MSYS, MinGW, or Git Bash?

Use tabs only in recipes so as to avoid confusing 'make'

Both of those approaches will fail on PowerShell/CMD. The Makefile can't properly support Windows without significant hacks. We would have to adapt build.zig and/or add a CMakeLists.txt file.

`prec(-1,...)` breaks `"word"` tokens

The reason for this might be that keywords, in most languages, cannot start with digits.