Skip to content

Conversation

@Mrtenz
Copy link
Member

@Mrtenz Mrtenz commented Mar 14, 2023

  • Exports have been renamed to be more generic.
    • ethErrors.rpc is now exported as rpcErrors.
    • ethErrors.provider is now exported as providerErrors. This is still Ethereum-specific.
  • Some related classes have been renamed to be more generic.
    • (Serialized)EthereumRpcError is now (Serialized)JsonRpcError.
  • References to eth-rpc-errors have been updated to rpc-errors.
    • We should rename the GitHub repository after merging this.

This is technically a breaking change, but we will be publishing this package under a new name.

@Mrtenz Mrtenz requested a review from a team as a code owner March 14, 2023 10:21
@Mrtenz Mrtenz force-pushed the mrtenz/rename-exports branch from d1c8ffc to 956c72e Compare April 11, 2023 13:28
Mrtenz added 3 commits April 12, 2023 11:31
…aming

This commit updates comments in the README file to make it clear that JSON-RPC errors and Ethereum EIP-1474 errors are under the 'rpcErrors' namespace, and adds that Ethereum EIP-1193 Provider errors are now under the 'providerErrors' namespace.
The providerErrors was not exported from the index.ts file, so it was imported separately from the errors module. This commit includes the export statement for providerErrors in the index.ts file.
Adjusting the coverage thresholds to meet the new requirements.
Copy link
Member

@FrederikBolding FrederikBolding left a comment

Choose a reason for hiding this comment

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

LGTM - good to merge when the repo name is updated.

@Mrtenz Mrtenz merged commit 53c754d into main Apr 12, 2023
@Mrtenz Mrtenz deleted the mrtenz/rename-exports branch April 12, 2023 11:45
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