Skip to content

chore: Cleaned up the supported chain#26

Merged
0xAtum merged 4 commits intomainfrom
leave-chain-config-to-users
Feb 10, 2025
Merged

chore: Cleaned up the supported chain#26
0xAtum merged 4 commits intomainfrom
leave-chain-config-to-users

Conversation

@0xAtum
Copy link
Owner

@0xAtum 0xAtum commented Feb 10, 2025

To keep the template lite, I've removed all the chains I put in. That way, the implementer can manually add the chains they want in Chains.json

  • Added Chains.json (script/config/Chains.json). Do not remove this file, otherwise the BaseScript will no longer work
  • Added Pre-loading in BaseScript. _loadContract(false) is being automatically called. If you want to load them in your simulation too, you will need to call _loadContract(true) in your run() function

@0xAtum 0xAtum merged commit b661a01 into main Feb 10, 2025
3 checks passed
@github-actions
Copy link

Slither report

THIS CHECKLIST IS NOT COMPLETE. Use --show-ignored-findings to show all the results.
Summary

missing-zero-check

Impact: Low
Confidence: Medium

constructor(address _owner) {

immutable-states

Impact: Optimization
Confidence: High

address public owner;

@0xAtum 0xAtum deleted the leave-chain-config-to-users branch February 10, 2025 13:39
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.

1 participant