Skip to content
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 immer dependency #972

Merged
merged 1 commit into from
Feb 6, 2024
Merged

Update immer dependency #972

merged 1 commit into from
Feb 6, 2024

Conversation

Baltoli
Copy link
Contributor

@Baltoli Baltoli commented Feb 6, 2024

In runtimeverification/immer#2, I made a tiny change to our immer dependency that will eventually enable us to update to C++20 when the time comes. This PR just bumps the backend submodule to use that new version.

Copy link
Collaborator

@dwightguth dwightguth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks like your update is changing slightly more than you expected. I'm not entirely sure this is correct. You might want to take a closer look at the exact commit history and the exact commit we were previously using.

@Baltoli
Copy link
Contributor Author

Baltoli commented Feb 6, 2024

Ah, well spotted - I presumed the submodule was up to date with the master branch of our fork but apparently not.

@rv-jenkins rv-jenkins merged commit b9aa5d8 into master Feb 6, 2024
7 checks passed
@rv-jenkins rv-jenkins deleted the update-immer branch February 6, 2024 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants