Skip to content

feat: fork scripts - #930

Merged
blablalf merged 38 commits into
mainfrom
feature/fork-scripts
Oct 2, 2025
Merged

feat: fork scripts#930
blablalf merged 38 commits into
mainfrom
feature/fork-scripts

Conversation

@blablalf

@blablalf blablalf commented Aug 26, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Rework of the fork deployment stack to inherit directly of the production scripts.
  • Add new scripts for SHORTDN & USDN + SHORTDN.

@blablalf blablalf self-assigned this Aug 26, 2025
blablalf
blablalf commented Sep 4, 2025
View reviewed changes

@blablalf blablalf left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Carefully auto-reviewed.

blablalf
blablalf commented Sep 4, 2025
View reviewed changes

@blablalf blablalf left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Doc reviewed.

@blablalf
blablalf marked this pull request as ready for review September 4, 2025 08:58
lilyanB
lilyanB reviewed Sep 11, 2025
View reviewed changes
Comment thread script/fork/ForkCore.s.sol Outdated
lilyanB
lilyanB reviewed Sep 11, 2025
View reviewed changes
Comment thread script/fork/ForkCore.s.sol
lilyanB
lilyanB reviewed Sep 19, 2025
View reviewed changes
Comment thread script/fork/README.md Outdated
@lilyanB

This comment was marked as resolved.

Sign in to view
lilyanB
lilyanB reviewed Sep 19, 2025
View reviewed changes
Comment thread script/fork/deployUsdnFork.sh
lilyanB
lilyanB reviewed Sep 19, 2025
View reviewed changes
Comment thread script/fork/deployUsdnFork.sh
lilyanB
lilyanB previously approved these changes Sep 19, 2025
View reviewed changes

@lilyanB lilyanB left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

deployments works on anvil
some comment, otherwise, it's good

@lilyanB

lilyanB commented Sep 23, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

@blablalf check these changes please: #936

And I approve just after the potential fix

@asyx21

asyx21 commented Sep 24, 2025 •
edited
Loading

Copy link
Copy Markdown

@blablalf check these changes: #936

renamed deployment scripts:

  • 01_DeployUsdnWstethUsd.s.sol → DeployUsdnWstethUsd.s.sol
  • 01_DeployUsdnWusdnEth.s.sol → DeployUsdnWusdnEth.s.sol

@blablalf

blablalf commented Oct 1, 2025

Copy link
Copy Markdown
Contributor Author

Ready for approval.

@blablalf
blablalf merged commit f22c0de into main Oct 2, 2025
6 checks passed
@blablalf
blablalf deleted the feature/fork-scripts branch October 2, 2025 07:52
@smardex-bot-account smardex-bot-account Bot mentioned this pull request Sep 29, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KirienzoEth KirienzoEth Awaiting requested review from KirienzoEth
@asyx21 asyx21 Awaiting requested review from asyx21
@aduezpro aduezpro Awaiting requested review from aduezpro
3 more reviewers
@evmd evmd evmd left review comments
@samooyo samooyo samooyo left review comments
@lilyanB lilyanB lilyanB approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@blablalf blablalf

Labels

2 reviews required

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@blablalf @lilyanB @asyx21 @evmd @samooyo @aduezpro