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

M1 Refactor, Movement SDK, Standalone Sui Execution, M2 as Movement SDK #63

Closed
wants to merge 17 commits into from

Conversation

l-monninger
Copy link
Collaborator

Pull Request

Description

  • This contribution represents a trivial change (skip the following sections).
  • This contribution represents a CI/CD change that needs to be tested e2e (skip the following sections; justify below).
    This PR incudes commits for
  • M1 Subnet Refactor (for code quality)
  • Development of Movement SDK (for internal organization)
  • Development of Standalone Sui Execution

Motivation and Context

Changes Made

  • Comprehensive changes to organization of M1, initially as an individual refactor, later as an migration to the Movement SDK.
  • Movement SDK framework crate added
  • Standalone Sui Execution developed against Movement SDK

Testing

  • Testing still in early stages
    • Initial subnet refactor appears to produce the same behavior

Screenshots (if applicable)

Related Issues

Checklist

  • I have read and followed the project's contributing guidelines.
  • I have tested the changes thoroughly.
  • My code follows the style and formatting guidelines of the project.
  • I have updated the relevant documentation, if applicable.
  • I have added tests, if appropriate.
  • The code builds without any errors or warnings.
  • All existing tests pass successfully.

Additional Notes

@l-monninger
Copy link
Collaborator Author

Daily checkpoint for Oct 16, 2023. Mainly Movement SDK and Sui planning.

@l-monninger
Copy link
Collaborator Author

Note that lint check with this PR will fail as its development preceded the adoption of the Conventional Commits standard.

@l-monninger
Copy link
Collaborator Author

canonical-new has reintroduced a lot of the above, but in a very different way. I am going to close this for now and not merge. However, outstanding elements of the M1 Refactor should likely be reincorporated from this branch at a later date.

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.

1 participant