Skip to content

Remove bridge#1815

Merged
Neopallium merged 4 commits intodevelopfrom
remove_bridge
Apr 17, 2025
Merged

Remove bridge#1815
Neopallium merged 4 commits intodevelopfrom
remove_bridge

Conversation

@Neopallium
Copy link
Copy Markdown
Contributor

changelog

modified external API

  • Removes the Bridge pallet storage.

data migration

  • Add migration to clear all storage with the Bridge pallet prefix.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

pallets/runtime/common/src/migration.rs:36

  • [nitpick] The error log message contains an emoji (🚨) that may not be suitable for production logging. Consider using plain text for a more professional output.
log::error!(
		"`clear_prefix` failed to remove all keys for {}. THIS SHOULD NEVER HAPPEN! 🚨",

pallets/runtime/common/src/migration.rs:44

  • [nitpick] The info log message includes an emoji (🧹) which could be considered unprofessional in production logs. It is recommended to remove emojis from log messages.
log::info!("Removed {} {} keys 🧹", keys_removed, P::get());

@Neopallium Neopallium merged commit 2137bc3 into develop Apr 17, 2025
17 checks passed
@Neopallium Neopallium deleted the remove_bridge branch April 17, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants