Skip to content

Improvements#1

Open
georgedigkas wants to merge 1 commit intomasterfrom
gd/improvements
Open

Improvements#1
georgedigkas wants to merge 1 commit intomasterfrom
gd/improvements

Conversation

@georgedigkas
Copy link
Collaborator

@georgedigkas georgedigkas commented Sep 2, 2025

Summary

Detail

Testing

  • 100% test coverage is maintained

Documentation


Requested Reviewers: @mention

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request modernizes the Move codebase by updating syntax to use the 2024 edition conventions. The main changes include migrating from the older module declaration syntax to the new syntax, updating dependency revisions to use "mainnet" instead of specific commit hashes, and removing test command options that are no longer available.

  • Updated module declarations from module name { ... } to module name; with use statements outside the braces
  • Changed Sui framework dependency revision from specific commit hash to "mainnet"
  • Simplified test command by removing --statistics and --coverage flags

Reviewed Changes

Copilot reviewed 23 out of 26 changed files in this pull request and generated no comments.

File Description
run.sh Simplified sui move test command by removing unsupported flags
packages/*/Move.toml Updated Sui dependency revision from commit hash to "mainnet"
packages//sources/.move Migrated to 2024 edition module syntax
packages//tests/.move Migrated to 2024 edition module syntax and improved formatting

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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