-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add support for extending providers #3932
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
Closed
Closed
Changes from 32 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
ea27270
Add a new extenders function for providers
nicosantangelo bc2ffd4
Delete the ExtendersManager class in favor of a simple array
nicosantangelo 5cd65ea
Delete package-lock.json
nicosantangelo 53cfffc
First approach to a lazy initialization provider
nicosantangelo b65ec8b
Clean up the code a bit and name functions
nicosantangelo 4c8cb3f
CustomError
nicosantangelo d37f1ad
Rename extenders dropping the 'Manager'
nicosantangelo 8f129a7
Rename internal extenders prop
nicosantangelo b6075a1
Be more specific on when the provider is initialized Lazily
nicosantangelo e18e61f
Enforce leading _ in private class properties
fvictorio b4f7749
Avoid floating promise in sendAsync
fvictorio 1dd01be
Fix strict-boolean-expression warnings
fvictorio 1e54d33
Allos Function type for EventEmitter functions
fvictorio 72b3295
Add UNINITIALIZED_PROVIDER error code
fvictorio 8c5f3f3
Create mean-crews-wait.md
fvictorio 2e58fe7
Make ProviderFactory type internal
nicosantangelo 32e7ae4
Allow for EventEmitter methods before initialization
nicosantangelo 5fdf636
Reuse type
nicosantangelo e24d580
Typos and better English
nicosantangelo 26becac
Forego of using sideeffects for creating providers
nicosantangelo 13a98a7
Docs
nicosantangelo 14252d1
LazyInitializationProvider tests
nicosantangelo 91a1659
construction tests
nicosantangelo eb5a8c6
Export the ProviderWrapper helper class
nicosantangelo 53fd5fa
Fix linting errors
nicosantangelo 3e4b413
Provider wrapper tests
nicosantangelo 8197e93
Supply the configuration to the extendProvider extenders
nicosantangelo fd17e6e
Supply the network name to the extendProvider extenders
nicosantangelo 6362de0
Add a way to access the wrapped provider and to initialize it
nicosantangelo 7d08583
Only initialize the provider once on init
nicosantangelo 83b0c48
Handle concurrent calls to init()
fvictorio 76948e4
Use done callback in sendAsync tests
fvictorio 4ac2e52
Rename LazyInitializationProvider to LazyInitializationProviderAdapter
nicosantangelo 6223bd6
Update comment
nicosantangelo 2d42b3a
Make this a minor change instead of patch
fvictorio a60b1cd
More explicit support for non-async provider extenders
nicosantangelo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "hardhat": patch | ||
| --- | ||
|
|
||
| Added support for extending the network provider |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
13 changes: 0 additions & 13 deletions
13
packages/hardhat-core/src/internal/core/config/extenders.ts
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.