Skip to content

Comments

Fixed warnings#50

Merged
arthurpaulino merged 1 commit intoargumentcomputer:mainfrom
imbrem:fix-warnings
Jan 31, 2025
Merged

Fixed warnings#50
arthurpaulino merged 1 commit intoargumentcomputer:mainfrom
imbrem:fix-warnings

Conversation

@imbrem
Copy link
Contributor

@imbrem imbrem commented Jan 31, 2025

Fixed deprecation warnings for latest Lean 4; shouldn't have changed any behavior or API surface

Copy link
Member

@arthurpaulino arthurpaulino left a comment

Choose a reason for hiding this comment

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

Thanks!

@arthurpaulino arthurpaulino merged commit 7f2c46b into argumentcomputer:main Jan 31, 2025
@imbrem
Copy link
Contributor Author

imbrem commented Jan 31, 2025

Thanks a lot! I'm trying to get Wasm.lean running on the latest version of Lean 4 so I can use it in my research, so I've just done a PR for the YatimaStdLib. Would you guys be fine updating that to run on the latest Batteries/Lean, or should I just maintain a fork?

@arthurpaulino
Copy link
Member

Let's talk on that PR. The short answer is that updating to latest Batteries/Lean is desirable.

@imbrem
Copy link
Contributor Author

imbrem commented Feb 1, 2025

@arthurpaulino I've already made the PR at argumentcomputer/YatimaStdLib.lean#100, and I've got PR's ready for straume, Megaparsec.lean and Wasm.lean ready to go once that's pushed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants