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

test(ECO-2887): Update loop test declaration per Aptos Labs guidance #632

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

alnoki
Copy link
Member

@alnoki alnoki commented Feb 19, 2025

See: aptos-labs/aptos-core#15713 (comment)

Description

Update the emojicoin arena Move unit tests per aptos-labs/aptos-core#15713 (comment)

Note that this does not affect the Move package, APIs, audit trail, etc. because it is only test code (no changes to source code)

Testing

From src/move/emojicoin_arena:

git ls-files | entr -c sh -c " \
       aptos move test --coverage --dev --move-2  &&
       aptos move fmt
   "

Copy link

vercel bot commented Feb 19, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
emojicoin-dot-fun ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2025 8:04pm
emojicoin-dot-fun-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2025 8:04pm

@alnoki alnoki enabled auto-merge February 19, 2025 20:07
@alnoki alnoki added this pull request to the merge queue Feb 19, 2025
Merged via the queue into arena with commit 465810e Feb 19, 2025
11 checks passed
@alnoki alnoki deleted the ECO-2887 branch February 19, 2025 20:19
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