-
Notifications
You must be signed in to change notification settings - Fork 103
Labels
good first issueGood for newcomersGood for newcomersrustIssues that affect or pull requests that update Rust codeIssues that affect or pull requests that update Rust code
Milestone
Description
PR #2191 made the errors module public and as a follow-up we should remove the public re-exports:
miden-base/crates/miden-objects/src/lib.rs
Lines 27 to 53 in 0f1ce86
| pub use errors::{ | |
| AccountDeltaError, | |
| AccountError, | |
| AccountIdError, | |
| AccountTreeError, | |
| AddressError, | |
| AssetError, | |
| AssetVaultError, | |
| AuthSchemeError, | |
| BatchAccountUpdateError, | |
| FeeError, | |
| NetworkIdError, | |
| NoteError, | |
| NullifierTreeError, | |
| PartialAssetVaultError, | |
| PartialBlockchainError, | |
| ProposedBatchError, | |
| ProposedBlockError, | |
| ProvenBatchError, | |
| ProvenTransactionError, | |
| StorageMapError, | |
| StorageSlotNameError, | |
| TokenSymbolError, | |
| TransactionInputError, | |
| TransactionOutputError, | |
| TransactionScriptError, | |
| }; |
This mostly requires fixing the error type imports in other places, which should be straightforward to do.
This can be done after #2191 is merged.
Context: #2191 (comment)
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomersrustIssues that affect or pull requests that update Rust codeIssues that affect or pull requests that update Rust code