Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename exports to be more generic #75

Merged
merged 5 commits into from
Apr 12, 2023
Merged

Rename exports to be more generic #75

merged 5 commits into from
Apr 12, 2023

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.

2 participants