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.

Sign up for GitHub

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

Jump to bottom

Deployment script #106

Merged
merged 29 commits into from
Oct 17, 2024
Merged

Deployment script #106

merged 29 commits into from
Oct 17, 2024

Conversation

sandstone-ag
Copy link
Contributor

No description provided.

@sandstone-ag sandstone-ag force-pushed the feature/deploy-script branch 2 times, most recently from 7b3da78 to af9fcff Compare September 11, 2024 07:42
@sandstone-ag sandstone-ag marked this pull request as ready for review September 11, 2024 07:44
@sandstone-ag sandstone-ag changed the title Add draft deploy script Deployment script Sep 11, 2024
@sandstone-ag
Add deploy script draft
9cc410a
@sandstone-ag
Move Lido addresses file
ee6f85c
@sandstone-ag
Use deployer address
09aed9b
@sandstone-ag
Implement switching between networks
46c90c0
@sandstone-ag
Merge changes from the develop branch
f3b8bb7
@sandstone-ag
Validate deployed contracts as a last step of the script
1059aba
@sandstone-ag
Implement missing methods in DeployValidation
95d8b81
@sandstone-ag
Make deploy validation errors more verbose
4375e39
@sandstone-ag
Various improvements
6ec1db4
@sandstone-ag
Various improvements
1256a57
@sandstone-ag
Rename some files
698a315
@sandstone-ag
Rename Deploy.s.sol to DeployBase.s.sol
2bb0c56
@sandstone-ag
Extract deploy main functionality to a separate abstract contract. Ad…
f8fce17 
…d separate deploy verification script.
@sandstone-ag
Add support for mocked Lido contracts on Holesky
70e43fa
@sandstone-ag
Add more checks to DeployVerification and config validation
0840b47
@sandstone-ag
Fix DeployConfig fields according to changed DGConfig fields
1b9c8b2
@sandstone-ag
Update forge-std library
c5e7c70
@sandstone-ag
Allow to read files from project dir. Update .env.example.
8ab73ed
@sandstone-ag
Move deploy config defaults to Config.sol
f17db4c
@sandstone-ag
Read deploy parameters from JSON file
8c6867e
@sandstone-ag
Use deploy script for scenario tests configuration
9640e8f
Psirex
Psirex reviewed Oct 16, 2024
View reviewed changes
scripts/deploy/Config.sol Show resolved Hide resolved
scripts/deploy/Config.sol Show resolved Hide resolved
scripts/deploy/ContractsDeployment.sol Show resolved Hide resolved
scripts/deploy/DeployBase.s.sol Show resolved Hide resolved
scripts/deploy/DeployVerification.sol Show resolved Hide resolved
scripts/deploy/DeployVerification.sol Outdated Show resolved Hide resolved
scripts/deploy/DeployVerification.sol Show resolved Hide resolved
scripts/deploy/JsonConfig.s.sol Show resolved Hide resolved
scripts/deploy/Readme.md Show resolved Hide resolved
Psirex
Psirex approved these changes Oct 17, 2024
View reviewed changes
Copy link
Contributor

@Psirex Psirex left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

@Psirex Psirex merged commit 518ee20 into develop Oct 17, 2024
2 checks passed
@sandstone-ag sandstone-ag deleted the feature/deploy-script branch October 17, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Psirex Psirex Psirex approved these changes

@rkolpakov rkolpakov Awaiting requested review from rkolpakov

@bulbozaur bulbozaur Awaiting requested review from bulbozaur

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sandstone-ag @Psirex