Skip to content

refactor(sequencer): remove unused asset storage variant#1704

Merged
SuperFluffy merged 2 commits intomainfrom
superfluffy/remove-unused-storage-variant
Oct 21, 2024
Merged

refactor(sequencer): remove unused asset storage variant#1704
SuperFluffy merged 2 commits intomainfrom
superfluffy/remove-unused-storage-variant

Conversation

@SuperFluffy
Copy link
Contributor

Summary

Removes an unused variant of the internal storage implementation of sequencer asset component.t

Background

As part of #1647 fees were moved from the assets to the fees component. The present variant was overlooked.

Changes

  • Remove a storage variant from the assets component.

Testing

Not applicable. Dead code.

Breaking Changelist

This is not a breaking change because a) the variant was never used (there was not even a theoretic way to write it after #1647) and b) no variants existed after the variant. If they had, this change would lead to a shift in the variants, causing breakage.

@SuperFluffy SuperFluffy requested a review from Fraser999 October 21, 2024 21:08
@SuperFluffy SuperFluffy requested a review from a team as a code owner October 21, 2024 21:08
@SuperFluffy SuperFluffy changed the title refactor(sequencer): remove unused storage variant refactor(sequencer): remove unused asset storage variant Oct 21, 2024
@github-actions github-actions bot added the sequencer pertaining to the astria-sequencer crate label Oct 21, 2024
@SuperFluffy SuperFluffy added this pull request to the merge queue Oct 21, 2024
Merged via the queue into main with commit 5ff422f Oct 21, 2024
@SuperFluffy SuperFluffy deleted the superfluffy/remove-unused-storage-variant branch October 21, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sequencer pertaining to the astria-sequencer crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants