Skip to content

Conversation

@rami3l
Copy link
Contributor

@rami3l rami3l commented Aug 14, 2025

Note

This PR was made by an LLM agent.

  • Fix documentation examples that use incorrect API calls like SnapshotArray::new() and Iter::of()
  • Replace SnapshotArray::new() with new() in documentation examples
  • Replace Iter::of([1, 2, 3]) with [1, 2, 3].iter() in documentation examples
  • Fix function name conflicts in benchmark files by renaming duplicate functions
  • Fix deprecated assert_*! syntax by removing exclamation marks
  • Fix function return type mismatches in documentation examples
  • Add let _ = for functions that return values but are unused in examples
  • Fix closure return types to match function signatures

All compilation errors have been resolved. The project now compiles successfully with only minor warnings remaining (mostly about unused trait implementations and formatting alignment).

@rami3l rami3l added the moonagent Created by MoonAgent. label Aug 14, 2025
@rami3l rami3l marked this pull request as draft August 21, 2025 10:23
@rami3l rami3l force-pushed the fix-compiler-warnings-20250814-041359 branch from b20dccc to 7a6cc9b Compare August 22, 2025 09:30
- Fix documentation examples that use incorrect API calls like `SnapshotArray::new()` and `Iter::of()`
- Replace `SnapshotArray::new()` with `new()` in documentation examples
- Replace `Iter::of([1, 2, 3])` with `[1, 2, 3].iter()` in documentation examples
- Fix function name conflicts in benchmark files by renaming duplicate functions
- Fix deprecated `assert_*!` syntax by removing exclamation marks
- Fix function return type mismatches in documentation examples
- Add `let _ = ` for functions that return values but are unused in examples
- Fix closure return types to match function signatures

All compilation errors have been resolved. The project now compiles successfully with only minor warnings remaining (mostly about unused trait implementations and formatting alignment).
@rami3l rami3l force-pushed the fix-compiler-warnings-20250814-041359 branch from 7a6cc9b to 0521230 Compare August 22, 2025 09:35
@rami3l rami3l force-pushed the fix-compiler-warnings-20250814-041359 branch from dd4b77c to d89c6c0 Compare August 25, 2025 02:19
@rami3l rami3l marked this pull request as ready for review August 25, 2025 02:20
/// inspect!(table.index(0).index, content="0")
/// }
///
/// ///|
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This deletion is unfortunate, but according to @bzy-debug there isn't a way to express panicking tests using the new syntax.

@rami3l rami3l merged commit ccf1582 into master Aug 25, 2025
1 check passed
@rami3l rami3l deleted the fix-compiler-warnings-20250814-041359 branch August 25, 2025 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

moonagent Created by MoonAgent.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant