-
Notifications
You must be signed in to change notification settings - Fork 4
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
Save analysis is gone, migrate to SCIP #9
Comments
Just wanted to come here to say I bumped into this when attempting to use this tool. |
Yeah you will have to use a pinned older nightly, until I find time to migrate to SCIP. It's obviously not a trivial effort. |
Yes, sorry if that came across as rude. This tool was perfect for my use case today. I can imagine it's not an easy task to migrate. Thanks a lot for your past work on this! |
@b-zee no need to worry, you weren't rude or anything. ❤️ |
Do you know which nightly release is the newest that still has save-analysis? |
I was so happy I found this crate, and then I saw it needed work to keep working. What does this migration entail? I don't think I have anything to offer, but I'd greatly benefit from having unused check across crates, so I want to at least subscribe to this. |
The algorithmic part of this crate is rather small, most of it is in the parsing of the save analysis. So it would amount to a rewrite really. |
Is there a somewhat detailed plan that splits the move to SCIP in smaller parts? I'd like to get involved. Finding dead code across crates sounds very useful to us :) |
save-analysis is going away. For the short to mid term future, we can pin nightlies but with increased MSRV requirements, this will cause issues.
Some more info on it in this reddit thread.
The text was updated successfully, but these errors were encountered: