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.

Sign up for GitHub

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

Jump to bottom

Use call_index attribute to fix warning and CI step build-all-benchmarks-packages #3557

Merged

RustNinja merged 2 commits into ComposableFi:main from RustNinja:fix-warning-add-call-index

May 4, 2023

Collaborator

RustNinja commented May 4, 2023 •

edited

Loading

use #[pallet::call_index(0)] for each callable function in pallet
it is by recommendation:

 Please ensure that all calls have the `pallet::call_index` attribute or that the
                     `dev-mode` of the pallet is enabled. For more info see:
                     <https://github.com/paritytech/substrate/pull/12891> and
                     <https://github.com/paritytech/substrate/pull/11381>.

it will fix CI step build-all-benchmarks-packages


          use call_index to fix warning and CI step build-all-benchmarks-packages

c19b6f2

RustNinja requested review from dzmitry-lahoda, blasrodri and kkast

May 4, 2023 12:45


          use #[pallet::call_index(0)]

3d250df

RustNinja mentioned this pull request

Add LP staking into Picasso runtime #3504

Merged

blasrodri approved these changes

View reviewed changes

RustNinja added this pull request to the merge queue

Merged via the queue into ComposableFi:main with commit 9f426fd

RustNinja deleted the fix-warning-add-call-index branch

May 4, 2023 15:38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet