-
Notifications
You must be signed in to change notification settings - Fork 225
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
feat: implement object deploy cmd for auto address derivation #343
Conversation
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThe changes introduce and modify command functionality for deploying Move packages. A new file ( Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant C as CLI Command
participant D as deploy()
participant TS as Transaction Service
U->>C: Run deployment command (full package or object)
C->>D: Invoke deploy(isObjectDeployment, verify)
alt Standard Deployment
D->>D: Validate package and module bundle
else Object Deployment
D->>D: Construct object address from parameters
D->>U: Request confirmation
U->>D: Confirm deployment
end
D->>TS: Construct and broadcast transaction
TS-->>D: Return transaction status
D-->>C: Return operation result
C-->>U: Display deployment outcome
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Files |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #343 +/- ##
=======================================
Coverage 41.15% 41.15%
=======================================
Files 269 269
Lines 25741 25741
=======================================
Hits 10593 10593
Misses 13504 13504
Partials 1644 1644 |
Description
Add object deployment cmd to help the users to derive object address.
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!
in the type prefix if API or client breaking changeReviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...