-
Notifications
You must be signed in to change notification settings - Fork 15
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
Update to Lean 4.8.0 #2
Conversation
Please can you resolve the conflicts? Feel free to take your resolutions over the ones I introduced with the 4.7.0 update. |
Nevermind, I did the merge for you keeping all your changes. Please take a look over the result, and clean up any diffs that no longer makes sense. Also, I think it would make more sense to bump to 4.8.0 (rather than the rc), or skip straight to 4.10.0 |
I also already did the merge, just haven’t pushed yet. |
Would you be okay pausing on further detail changes? I’m a bit worried I have to redo all of my merge now. |
I have a lot of downstream changes as well. |
Actually let's hold off on the 4.10 upgrade, since I have a few more PRs to get in after this, and merging them prior to the upgrade will be easier. |
On v4.8.0 exactly now, rather than -rc2. |
No description provided.