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

Refactor pallet-proxy to use v2 benchmarks #6211

Conversation

programskillforverification
Copy link
Contributor

Description

This PR moves the pallet-proxy to the v2 of frame_benchmarking.

Integration

N/A

@programskillforverification
Copy link
Contributor Author

@ggwpez Could you add R0 label? We have many similar pallet to refactor v2, and their prdocs are similar.

@programskillforverification programskillforverification changed the title Refactor pallet-grandpa to use v2 benchmarks Refactor pallet-proxy to use v2 benchmarks Oct 25, 2024
@programskillforverification
Copy link
Contributor Author

@ggwpez pls review

@gui1117 gui1117 added the T2-pallets This PR/Issue is related to a particular pallet. label Oct 26, 2024
@gui1117 gui1117 added the R0-silent Changes should not be mentioned in any release notes label Oct 26, 2024
@shawntabrizi
Copy link
Member

@programskillforverification why was this closed?

@programskillforverification
Copy link
Contributor Author

@programskillforverification why was this closed?

#5995 has done the same thing with this pr

@shawntabrizi
Copy link
Member

Ah, I am sorry about that.

your work here though is good, so if you would like to receive a small tip in DOT we can still do that for you.

@programskillforverification
Copy link
Contributor Author

Ah, I am sorry about that.

your work here though is good, so if you would like to receive a small tip in DOT we can still do that for you.

Thanks a lot. I am uncomfortable to receive tip due to not merge. I will pick new pallet and continue to contribute.

@shawntabrizi
Copy link
Member

@programskillforverification okay! thank you for your continuing contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T2-pallets This PR/Issue is related to a particular pallet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants