-
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
adding back permission perf test #15765
Conversation
⏱️ 3h 34m total CI duration on this PR
🚨 2 jobs on the last run were significantly faster/slower than expected
|
bf8f126
to
684590e
Compare
684590e
to
1bbb138
Compare
1bbb138
to
95d2bf9
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
95d2bf9
to
48b4403
Compare
APTTransferWithPermissionedSigner 34 0.946 1.091 1068.0 | ||
APTTransferWithMasterSigner 34 0.946 1.091 185.0 |
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.
The tab-delimited format used throughout this file appears to be inconsistent in these two new lines. To maintain consistency, the spaces between fields should be replaced with tab characters. The lines should be formatted as:
APTTransferWithPermissionedSigner 34 0.946 1.091 1068.0
APTTransferWithMasterSigner 34 0.946 1.091 185.0
Spotted by Graphite Reviewer
Is this helpful? React 👍 or 👎 to let us know.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
48b4403
to
2c8f58e
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ Forge suite
|
✅ Forge suite
|
Description
How Has This Been Tested?
Key Areas to Review
Type of Change
Which Components or Systems Does This Change Impact?
Checklist