-
Notifications
You must be signed in to change notification settings - Fork 144
Public dependency on once_cell
#575
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
I wonder if anyone ever actually uses |
Also see #512. I'm not against releasing a new major version, but then I think we should try to fix the issues listed in https://github.com/trishume/syntect/milestone/1 and maybe go through new issues since then and see if there is more to do. It would be nice to not have to release 7.0 shortly after 6.0. |
Well since it sounds like everyone is on board with getting rid of I can also spend some of my freetime over the next couple of months working towards the 6.0 milestone. #512 does look like a good place to start |
Now that there are things like
OnceLock
(since 1.70.0) andLazyLock
(since 1.80.0) in the standard library it'd be nice to migrate off ofonce_cell
. The one current issue with this is that it's a breaking change for this library due tosyntect::parsing::SCOPE_REPO
being a part of the public API while it's aonce_cell::sync::Lazy
. It'd be nice to change this in the next breaking release if possible (and I'd be willing to open a PR to handle it)The text was updated successfully, but these errors were encountered: