Skip to content

Conversation

@huyhuynh3103
Copy link
Contributor

Description

This pull request introduces significant changes to the repository, including updates to workflows, dependency management, configuration files, and the addition of new scripts while removing older ones. The changes aim to modernize the project setup, enhance automation, and simplify dependency management. Below is a summary of the most important changes:

Workflow and Build Process Updates:

  • Updated .github/workflows/test.yml to refine branch matching patterns and replace the nightly version of Foundry with soldeer for dependency management. Additionally, the forge build command was simplified by removing the --sizes flag. [1] [2]

Dependency Management:

  • Removed Git submodules (forge-std, openzeppelin-contracts, solady, contract-libs) from .gitmodules and replaced them with direct dependencies in foundry.toml using soldeer. This simplifies dependency updates and eliminates the need for submodule management. [1] [2]
  • Added configuration for soldeer in foundry.toml to manage remappings and recursive dependencies.

New Scripts and Automation:

  • Added generate-artifact.sh to automate artifact generation with extensive support for command-line arguments and JSON output.
  • Introduced install.sh to automate the installation of Foundry, Rust, and soldeer, ensuring the environment is properly set up.
  • Enhanced run.sh with additional options for deployment scripts, including support for artifact generation, address exporting, and contract verification.

Configuration and Formatting:

  • Updated foundry.toml with new formatting rules (e.g., sorted imports, wrapped comments) and switched the EVM version from istanbul to london. The libs directory was also renamed to dependencies.

Code Cleanup:

  • Removed legacy Solidity files (ArtifactFactory.sol, BaseGeneralConfig.sol) and associated dependencies, replacing them with updated configurations and scripts. [1] [2]

These changes collectively modernize the repository, streamline the build and deployment process, and improve maintainability.

Checklist

  • I have clearly commented on all the main functions following the NatSpec Format
  • The box that allows repo maintainers to update this PR is checked
  • I tested locally to make sure this feature/fix works

TuDo1403 added 30 commits May 16, 2024 11:16
…mp-package

feat(rework): implement `bump-package`
…er-defined-config

feat(rework): implement `user-defined-config`
tu-do.ron and others added 27 commits August 27, 2024 17:30
…d-initialize-guard

feat(rework): implement `add-initialize-guard`
…d-default-contracts

feat(rework): implement `add-default-contracts`
…d-profile

feat(rework): implement `add-profile`
…store-transparent-proxy-v4_9_5

feat(rework): implement `restore-transparent-proxy-v4_9_5`
…adapt new forge version when generating storage layout
Co-authored-by: HuyHuynh <63286199+huyhuynh3103@users.noreply.github.com>
…nitializer

fix: add --json when inspecting methodIdentifiers
BREAKING_CHANGE: adapt soldeer, enhance script runtime, extra features
@huyhuynh3103 huyhuynh3103 requested a review from TuDo1403 August 4, 2025 08:37
@huyhuynh3103 huyhuynh3103 merged commit 3c90e96 into mainnet Aug 4, 2025
3 checks passed
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.

3 participants