Skip to content

Conversation

@rami3l
Copy link

@rami3l rami3l commented Aug 18, 2025

Note

This PR was made by an LLM agent.

  • Replace deprecated f?(..) syntax with try? f(..) syntax
  • Replace deprecated char_at() calls with get_char().unwrap()
  • Replace deprecated @math.minimum with @cmp.minimum
  • Remove unused trait implementations (Default for Cirru and Show derive)
  • Make "js" the project's preferred build target

All compiler warnings and errors have been resolved while maintaining code functionality.

moonagent and others added 2 commits August 18, 2025 06:32
- Replace deprecated f?(..) syntax with try? f(..) syntax
- Replace deprecated char_at() calls with get_char().unwrap()
- Replace deprecated @math.minimum with @cmp.minimum
- Remove unused trait implementations (Default for Cirru and Show derive)
- Make "js" the project's preferred build target

All compiler warnings and errors have been resolved while maintaining code functionality.
@tiye
Copy link
Member

tiye commented Aug 19, 2025

this is cool. however I prefer rewrite my code locally. MoonBit syntax updated again?

@rami3l
Copy link
Author

rami3l commented Aug 21, 2025

this is cool. however I prefer rewrite my code locally. MoonBit syntax updated again?

@tiye This PR seems to be mostly about replacing deprecated APIs, and you are free to edit it as you wish. Regarding syntax changes, most changes are actually in .mbti files since indeed the syntax of these files are still unstable.

@tiye tiye closed this Aug 30, 2025
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