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

Remove pallet::getter macro from pallet-identity #4586

Merged

Conversation

PolkadotDom
Copy link
Contributor

As per #3326, removes pallet::getter macro usage from the pallet-identity. The syntax StorageItem::<T, I>::get() should be used instead.

Also makes all storage values public

cc @muraca

@PolkadotDom PolkadotDom requested a review from a team as a code owner May 25, 2024 15:37
@bkchr bkchr added the T1-FRAME This PR/Issue is related to core FRAME, the framework. label May 26, 2024
@bkchr
Copy link
Member

bkchr commented May 26, 2024

@PolkadotDom ci is not happy

reduce kitchensink bump to patch

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6316716

@PolkadotDom PolkadotDom requested a review from ggwpez May 28, 2024 16:28
prdoc/pr_4586.prdoc Outdated Show resolved Hide resolved

crates:
- name: pallet-identity
bump: major
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It also needs to mention pallet-alliance with a patch, since it was modified.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xlc getting conflicting comments here. @ggwpez In a comment above was told to remove the semver bumps since they don't change in a 'semver' sense.

I'd say I agree with xlc since there's really no functionality added/bugs fixed, but I'm happy to do whatever!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is not modified. the tests are not published

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hadn't seen there was a none option. Have set the bumps to that.

prdoc/pr_4586.prdoc Outdated Show resolved Hide resolved
@PolkadotDom PolkadotDom requested a review from ggwpez July 6, 2024 17:46
@bkchr
Copy link
Member

bkchr commented Jul 8, 2024

@PolkadotDom this branch has conflicts.

@bkchr
Copy link
Member

bkchr commented Jul 23, 2024

bot fmt

@command-bot
Copy link

command-bot bot commented Jul 23, 2024

@bkchr https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6788844 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/fmt/fmt.sh". Check out https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 1-be572ffd-1eaa-42d4-a11a-ab265ebe94d0 to cancel this command or bot cancel to cancel all commands in this pull request.

Copy link

We are migrating the command bot to be a GitHub Action

Please, see the documentation on how to use it

@command-bot
Copy link

command-bot bot commented Jul 23, 2024

@bkchr Command "$PIPELINE_SCRIPTS_DIR/commands/fmt/fmt.sh" has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6788844 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6788844/artifacts/download.

@bkchr bkchr enabled auto-merge July 23, 2024 16:53
@bkchr bkchr added this pull request to the merge queue Jul 23, 2024
Merged via the queue into paritytech:master with commit 9974a68 Jul 23, 2024
158 of 161 checks passed
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
As per paritytech#3326, removes pallet::getter macro usage from the
pallet-identity. The syntax `StorageItem::<T, I>::get()` should be used
instead.

Also makes all storage values public

cc @muraca

---------

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Bastian Köcher <git@kchr.de>
Co-authored-by: command-bot <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T1-FRAME This PR/Issue is related to core FRAME, the framework.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants