Skip to content

Update tree-sitter and grammars to version 0.20 #674

Description

@marco-c
No description provided.

Activity

  1. dburriss commented on Mar 20, 2022

    @dburriss
    Contributor

    Based on my issues adding tree-sitter-c-sharp I thought I would do a little look at the feasibility of rolling forward to 0.20.

    • tree-sitter-java (version = "0.19.0" | tree-sitter = "0.19" | PR)
    • tree-sitter-typescript (version = "0.20.0" | tree-sitter = ">= 0.19, < 0.21")
    • tree-sitter-javascript (version = "0.20.0" | tree-sitter = ">= 0.19, < 0.21")
    • tree-sitter-python (version = "0.19.0" | tree-sitter = ">= 0.19, < 0.21" | PR)
    • tree-sitter-rust (version = "0.20.1" | tree-sitter = "0.20" )

    For c-sharp there is a PR mentioned in #789 by the maintainer.

  2. marco-c commented on Mar 21, 2022

    @marco-c
    CollaboratorAuthor

    The PRs are being ignored by the maintainers, we could use forks and specify dependencies as git repositories (https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-git-repositories).
    For example, for tree-sitter-python, we could specify tree-sitter-python = { git = "https://github.com/Luni-4/tree-sitter-python", branch = "update-0.20" }.

  3. dburriss commented on Apr 7, 2022

    @dburriss
    Contributor

    @marco-c I was looking at #812 and was wondering if we have a final action plan here?
    That PR looks like it is uncovering some good bugs uncovered by the upgrade (or some that were there already?).

    I did notice Python points to its tree-sitter master and the other to a fork from @BrainBuzzer that is a work in progress.
    In the merge, this would point to something more stable than master though.

    I do want to ask though, what does 0.20 give us? Were there important security fixes, perf, features, that the project benefited from? Couldn't we settle on 0.19 until the situation improves with dependencies? This only makes sense if we think the situation CAN be improved. If it isn't something any of us can influence, then the forking strategy is the way to go.

  4. linked a pull request that will close this issueUpdate grammars to 0.20 #954on Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Fields

Priority

None yet

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions