-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[framework] Add any_map::AnyMap #15574
base: igor/btree_map
Are you sure you want to change the base?
Conversation
⏱️ 54m total CI duration on this PR
🚨 1 job on the last run was significantly faster/slower than expected
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
@@ -53,5 +53,8 @@ module aptos_std::any { | |||
fun test_any() { | |||
assert!(unpack<u64>(pack(22)) == 22, 1); | |||
assert!(unpack<S>(pack(S { x: 22 })) == S { x: 22 }, 2); | |||
|
|||
assert!(unpack<S>(pack(S { x: 22 })) == S { x: 22 }, 2); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this line as it uses incorrect method call syntax for unpack. The assertion on line 56 already tests the same functionality with correct function call syntax.
Spotted by Graphite Reviewer (based on CI logs)
Is this helpful? React 👍 or 👎 to let us know.
1457de6
to
6845566
Compare
6845566
to
3013155
Compare
4024942
to
6f26c8b
Compare
3013155
to
58ebe00
Compare
6f26c8b
to
0eae837
Compare
58ebe00
to
6d5e10b
Compare
0eae837
to
ca43d53
Compare
6d5e10b
to
be8233a
Compare
ca43d53
to
6cd4fcf
Compare
be8233a
to
d5c471a
Compare
6cd4fcf
to
79ceaf7
Compare
d5c471a
to
44d2587
Compare
79ceaf7
to
c13fc13
Compare
44d2587
to
2a948ea
Compare
c13fc13
to
ed8de0e
Compare
2a948ea
to
d311703
Compare
ed8de0e
to
cf1e96f
Compare
d311703
to
c292f09
Compare
cf1e96f
to
fa85df7
Compare
c292f09
to
0ff3f3d
Compare
fa85df7
to
726f868
Compare
0ff3f3d
to
7295cea
Compare
9b16865
to
f5cf9c7
Compare
f5cf9c7
to
cbb63e1
Compare
7295cea
to
c54c812
Compare
cbb63e1
to
49924d1
Compare
Description
How Has This Been Tested?
Key Areas to Review
Type of Change
Which Components or Systems Does This Change Impact?
Checklist