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

Refactor sp-mvm/tests/assets.rs #108

Merged
merged 1 commit into from
Oct 8, 2021
Merged

Refactor sp-mvm/tests/assets.rs #108

merged 1 commit into from
Oct 8, 2021

Conversation

vitvakatu
Copy link
Contributor

A bunch of const variables and obscure enum -> array index conversions
were removed and replaced with Asset and Package structs. All modules
and transactions are available in a separate modules as static
variables. They are not Copy, so passing by reference is required.

Binary data is loaded in runtime using once_cell crate. It allows to
avoid recompilation on changes in assets.

A bunch of const variables and obscure enum -> array index conversions
were removed and replaced with Asset and Package structs. All modules
and transactions are available in a separate modules as static
variables. They are not Copy, so passing by reference is required.

Binary data is loaded in runtime using once_cell crate. It allows to
avoid recompilation on changes in assets.
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1288366527

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 53.942%

Totals Coverage Status
Change from base Build 1214784954: 0.0%
Covered Lines: 97
Relevant Lines: 159

💛 - Coveralls

@borispovod borispovod self-requested a review October 4, 2021 15:08
Copy link
Member

@borispovod borispovod left a comment

Choose a reason for hiding this comment

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

👍

@borispovod borispovod merged commit 17ab094 into master Oct 8, 2021
@boozook boozook deleted the refactor_assets branch December 6, 2021 14:16
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.

3 participants