-
-
Notifications
You must be signed in to change notification settings - Fork 95
Use typed data APIs #424
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
Use typed data APIs #424
Conversation
Untyped data APIs are deprecated.
|
Thank you for the PR, but, the version of the code you are targeting is years old. Where is it being used? |
|
Please help me understand. What is
That comment was from a year ago. Has anyone tried updating since then? |
|
It looks like an older version of the lobste.rs source code, used to benchmark YJIT. Lobsters does use Commonmarker 2.x now, since July this year. Maybe the benchmark could be updated rather than relying on years-old code. |
|
Thank you, we'll update or remove the old gem. |
|
Thank you! |
|
FWIW we're still using the c-api-stable branch at GitHub, and it'd be handy to have this change, but I also understand if you no longer want to support this old branch. If you are still open to supporting it, I could also open a PR to relax the Ruby version constraint on that branch along the same lines as 5655513 Thank you! |
|
What's the apprehension to upgrading off of a four-year-old branch? |
|
Mainly staffing (failing tests and nobody to fix them) but also general avoidance of rust extensions for various reasons. Neither of those are your problem at all, so I fully understand whatever maintenance choice you make. |
|
Well, mostly, I'm just trying to understand the apprehension, as I said. If I need to keep maintaining a branch I'm not interested in maintaining, I'd like a good reason. Recently, someone made the case that a change needed to be made to fix an issue with GitHub Pages, so I fixed it. But in this thread, the reasons have been "there's a random outdated benchmark somewhere" and "an avoidance of rust extensions," neither of which are actual explanations for me to understand the problem of why this change needs to be made, what problems it solves, what problems not fixing it causes, etc. |
Untyped data APIs are deprecated.