-
Notifications
You must be signed in to change notification settings - Fork 221
[Epic] Pure C API #2133
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
Comments
It looks like all the linked PRs have been merged. Is this done? Is there a way to get a prerelease gem pushed for testing, or JRuby added to CI? |
The main missing part would be #2398. |
@Morriar Ok thank you for that update! In #2433 I noticed there are still a number of places that the I am standing by to help. The current dependencies prevent doing much on my end, though. |
With #2398, Here's how we use it from Sorbet (C++ implementation): https://github.com/sorbet/sorbet/blob/master/rbs/rbs_common.h#L5 and we don't need the Ruby VM. |
Uh oh!
There was an error while loading. Please reload this page.
Shopify is actively working on decoupling the RBS parser from its Ruby bindings, in order to expose a new pure C API.
This API will be more compatible (e.g. it can integrate into JRuby, Truffle, Sorbet, etc.), and will allow for multi-threaded RBS parsing (since it won't be blocked by Ruby's GVL).
PR list
This is still a work-in-progress, but this list will be updated to list all PRs that are ready to review, in the order that they should be merged.
.c
/.h
files #2098CC @Morriar, @soutaro
The text was updated successfully, but these errors were encountered: