Skip to content
This repository has been archived by the owner on May 4, 2024. It is now read-only.

Update BasicCoin.move #1050

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update BasicCoin.move #1050

wants to merge 1 commit into from

Conversation

gftea
Copy link

@gftea gftea commented May 13, 2023

Fix warnings

warning[W10007]: potential issue with attribute value
   ┌─ ./sources/BasicCoin.move:94:24
   │
94 │     #[expected_failure(abort_code = 2)] // Can specify an abort code
   │                        ^^^^^^^^^^   - Replace value with constant from expected module or add `location=...` attribute.
   │                        │             
   │                        WARNING: passes for an abort from any module.

Motivation

(Write your motivation for proposed changes here.)

Have you read the Contributing Guidelines on pull requests?

(Write your answer here.)

Test Plan

(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)

Fix warnings
```
warning[W10007]: potential issue with attribute value
   ┌─ ./sources/BasicCoin.move:94:24
   │
94 │     #[expected_failure(abort_code = 2)] // Can specify an abort code
   │                        ^^^^^^^^^^   - Replace value with constant from expected module or add `location=...` attribute.
   │                        │             
   │                        WARNING: passes for an abort from any module.
```
@gftea
Copy link
Author

gftea commented May 15, 2023

@wrwg wonder if anyone can review and approve workflow run

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant