-
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
Update release.yaml #10679
Update release.yaml #10679
Conversation
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.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
title: "AIP-55: Generalize Transaction Authentication and Support Arbitrary K-of-N MultiKey Accounts" | ||
description: "AIP-55: Generalize Transaction Authentication and Support Arbitrary K-of-N MultiKey Accounts." | ||
discussion_url: "https://github.com/aptos-foundation/AIPs/issues/267" | ||
execution_mode: MultiStep | ||
update_sequence: | ||
- FeatureFlag: | ||
enabled: | ||
- bulletproofs_natives | ||
- name: step_6_enable_storage_deletion_refund | ||
- single_sender_authenticator |
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.
should we combine this was the secp? since it is the same feature flag.
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.
@runtian-zhou also confirmed that they should be combined if they use the same feature flag.
By the way, is this a proper way to combine the information of the two steps? https://github.com/aptos-labs/aptos-core/pull/10679/files#diff-f0942a7d2fc980b7d9d90ec193ec4e9214cdb34c3e3c1d59fa7d5507325b1574R15
I am not sure if discussion_url: https://url1, https://url2
will end up generating a proper link to the discussions? (@movekevin , @perryjrandall , @runtian-zhou)
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.
@kent-white confirmed that two discussion urls in one proposal would not work in UI. So, I added the links to the description
field and kept only one link (for AIP-55) in the discussion_url
field.
@davidiw , @movekevin , @perryjrandall , @runtian-zhou , please let me know if you have any comments or concern for this.
5168d75
to
813b924
Compare
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.
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.
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.
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.
All look correct
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.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
95e5152
to
b07fbf7
Compare
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.
❌ Forge suite
|
✅ Forge suite
|
✅ Forge suite
|
Description
Update
release.yaml
for v1.8