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

Prism added node_id and Node#breadth_first_search in the 1.0 release.… #49

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

HParker
Copy link
Contributor

@HParker HParker commented Sep 18, 2024

Prism added node_id and Node#breadth_first_search in the 1.0 release. These methods are required for Prism to be able to find the method from the backtrace.

https://github.com/ruby/prism/blob/main/CHANGELOG.md#100---2024-08-28

In practice you will likely only end up in this situation if you previously had pre-1.0 prism installed and upgrade Ruby to a version with Prism as the default parser, but haven't upgraded Prism yet.

However, it is probably worth avoiding this error even for this very rare case.

cc: @kddnewton

… These methods are required for Prism to be able to find the method from the backtrace.

https://github.com/ruby/prism/blob/main/CHANGELOG.md#100---2024-08-28

In practice you will likely only end up in this situation if you previously had pre-1.0 prism installed and upgrade Ruby to a version with Prism as the default parser.
@HParker HParker force-pushed the increase-Prism-version-requirement branch from f5e0f73 to cb574da Compare September 18, 2024 23:08
@kddnewton kddnewton merged commit 9368229 into ruby:master Sep 19, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants