-
Notifications
You must be signed in to change notification settings - Fork 57
Updated to @chainlink/[email protected], @chainlink/[email protected], @chainlink/[email protected] #40
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
base: main
Are you sure you want to change the base?
Conversation
- Bumped to [email protected] - Bumped to [email protected] - Bumped to local@[email protected] - Added preprocessor - Added remappings - Updated import paths to account for unvendored OZ contracts
- Updated no-fork tests to use fully qualified name for MockCCIPRouter due to multiple artifacts originating from both chainlink/contracts-ccip and chianlink/local - not ordering of imports in hardhat.config.ts
- Added declaration file for remappings-helper - Relocated remappings-helper - Updated import path
- Moved hardhat-preprocessor to dev dependency - Moved OpenZeppelin to dev dependency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - note comments and questions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make TS file?
Also if there is a canonical reference for this pattern (maybe in the official docs or the preprocess NPM package) maybe we can add that to the multi line comment on top?
"@chainlink/local": "0.2.7-beta.0", | ||
"@nomicfoundation/hardhat-toolbox": "^5.0.0", | ||
"hardhat": "^2.22.7" | ||
"hardhat": "^2.22.7", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need to update README to say that this repo uses remappings for HH2 (similar to this README: https://github.com/smartcontractkit/hardhat-starter-kit)?
Are we doing HH3 branch here too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Redundant file given that hardhat version used in this PR is 2.22.7
Summary
This PR updates several dependencies and project configurations to align with the latest versions of Chainlink packages and improve local development stability.
Dependency Updates
chainlink/[email protected]
chainlink/[email protected]
chainlink/[email protected]
Project Configuration
remappings-helper
declaration file and relocated it for better organizationMockCCIPRouter
(due to duplicate artifacts inchainlink/contracts-ccip
andchainlink/local
)Changes
test/no-fork/Example1.spec.ts
test/no-fork/Example2.spec.ts
contracts/BasicTokenSender.sol
contracts/ProgrammableTokenTransfers.sol
contracts/utils/Dependencies.sol
contracts/utils/Withdraw.sol
hardhat.config.ts
helpers/remappings/remappings-helper.d.ts
helpers/remappings/remappings-helper.js
remappings.txt
package.json
package-lock.json
All other changes are a result of dependency upgrades.