`AssetInfoBase<T>` should be non-exhaustive

mars-protocol / cw-asset

A unified representation of various types of Cosmos fungible assets, and helper functions for interacting with them

Apache License 2.0

27 stars 13 forks source link

`AssetInfoBase<T>` should be non-exhaustive #14

Closed y-pakorn closed 2 years ago

y-pakorn commented 2 years ago

Since the addition of Cw1155 variants of AssetInfoBase, we should consider adding the #[non_exhaustive] macro tag to prevent breaking change of pattern matching when adding new enum variants.

larry0x commented 2 years ago

closing this issue as it is implemented in #16