Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pin tree-sitter to the 0.20.10 release #8396

Merged
merged 1 commit into from
Oct 25, 2023
Merged

Pin tree-sitter to the 0.20.10 release #8396

merged 1 commit into from
Oct 25, 2023

Conversation

the-mikedavis
Copy link
Member

Packagers downstream sometimes struggle with git dependencies so we want to switch to the published crate for the 23.09 (Helix) release. We can revert this and update the pinned commit after the release.

@the-mikedavis the-mikedavis added A-dependencies Area: Dependency rust Pull requests that update Rust code labels Sep 26, 2023
@the-mikedavis the-mikedavis added this to the 23.9 milestone Sep 26, 2023
@David-Else
Copy link
Contributor

Sorry for my ignorance, but doesn't this mean the new Helix release will have an inferior 6 month old version of tree sitter? This Helix version will be the version the vast majority of people will be using for the next 3 or so months, so it would be quite a big deal?

@the-mikedavis
Copy link
Member Author

The only issue I know of with 0.20.10 that affects us is #7689 so I don't see this being a big deal

@pascalkuthe
Copy link
Member

And we have been blocking the helix release on TS releasing for close to two months now. At some point, we just have to stop waiting. Once TS releases its next version we can do a patch release.

@David-Else
Copy link
Contributor

tree-sitter/tree-sitter#2669 (comment)

at the end of this week there will be a new tree-sitter release so I'll include this request in it.

@archseer
Copy link
Member

Now a month ago. We can't wait any longer so we'll merge this revert

@archseer archseer merged commit 2ebcc4d into master Oct 25, 2023
6 checks passed
@archseer archseer deleted the pin-ts-to-release branch October 25, 2023 16:08
archseer added a commit that referenced this pull request Oct 26, 2023
We only reverted so that the latest release would use a stable
tree-sitter version hosted on crates.io. We do want the improvements
on nightly.

This reverts commit 2ebcc4d.
danillos pushed a commit to danillos/helix that referenced this pull request Nov 21, 2023
danillos pushed a commit to danillos/helix that referenced this pull request Nov 21, 2023
We only reverted so that the latest release would use a stable
tree-sitter version hosted on crates.io. We do want the improvements
on nightly.

This reverts commit 2ebcc4d.
dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
We only reverted so that the latest release would use a stable
tree-sitter version hosted on crates.io. We do want the improvements
on nightly.

This reverts commit 2ebcc4d.
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
We only reverted so that the latest release would use a stable
tree-sitter version hosted on crates.io. We do want the improvements
on nightly.

This reverts commit 2ebcc4d.
Vulpesx pushed a commit to Vulpesx/helix that referenced this pull request Jun 7, 2024
Vulpesx pushed a commit to Vulpesx/helix that referenced this pull request Jun 7, 2024
We only reverted so that the latest release would use a stable
tree-sitter version hosted on crates.io. We do want the improvements
on nightly.

This reverts commit 2ebcc4d.
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
We only reverted so that the latest release would use a stable
tree-sitter version hosted on crates.io. We do want the improvements
on nightly.

This reverts commit 2ebcc4d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-dependencies Area: Dependency rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants