Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

benchmark cli: pallet list returns instance names #14757

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

muharem
Copy link
Contributor

@muharem muharem commented Aug 11, 2023

benchmark pallet --list returns instance names, result <pallet name>, <extrinsic>, <instance name>.

questions:

  • can this break anything? because of this risk the instance name appears at the end, even though it looks more logical to have it after the name
  • some names resolves as SystemBench::<Runtime>, there is also an example how to fix it

@muharem
Copy link
Contributor Author

muharem commented Aug 14, 2023

or we can put the new field under some --output flag

@gilescope gilescope requested a review from chevdor August 15, 2023 12:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant