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

Add links to deploy script #225

Merged

filmakarov merged 1 commit into dev from chore/update-readme-for-deploy

Dec 10, 2024

Collaborator

filmakarov commented Dec 10, 2024

Add links to deploy script


          deploy

bc78319

filmakarov merged commit b827f46 into dev

5 checks passed

filmakarov deleted the chore/update-readme-for-deploy branch

December 10, 2024 09:20

github-actions bot commented Dec 10, 2024

🤖 Slither Analysis Report 🔎

Slither report

# Slither report

THIS CHECKLIST IS NOT COMPLETE. Use --show-ignored-findings to show all the results.
Summary
🟡 - locked-ether (1 results) (Medium)

constable-states (2 results) (Optimization)

locked-ether

🟡 Impact: Medium
🔴 Confidence: High

ID-0
Contract locking ether found:
Contract NexusBootstrap has payable functions:
- ModuleManager.receive()
- ModuleManager.fallback(bytes)
  But does not have a function to withdraw the ether

utils/NexusBootstrap.sol#L33-L165

constable-states

Impact: Optimization
🔴 Confidence: High

ID-1
RegistryAdapter.registry should be constant

base/RegistryAdapter.sol#L10

ID-2
RegistryFactory.threshold should be constant

factory/RegistryFactory.sol#L39

_This comment was automatically generated by the GitHub Actions workflow._

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet