Skip to content

Conversation

@d4rm5
Copy link
Collaborator

@d4rm5 d4rm5 commented Aug 26, 2025

This pull request introduces a major restructuring of the devcontainer setup, along with significant improvements to the CI workflow and development environment.

Devcontainer Restructuring

  • Multiple Devcontainer Variants: The project now supports multiple devcontainer variants within the .devcontainer directory. This allows for different development
    environments tailored to specific needs (e.g., auditor, hardened, isolated, minimal, legacy).
  • Comprehensive Documentation: The README has been updated with comprehensive documentation for each devcontainer variant, explaining their purpose and included tools.

CI Workflow Enhancements

  • Tiered Testing: The GitHub Actions workflow for testing the devcontainers has been updated to use a tiered approach:
    • A "Minimal Tools" test runs on all containers to verify baseline functionality (e.g., Foundry).
    • An "Auditor Tools" test runs on auditor, hardened, and isolated containers to check for specialized tools like Slither, Mythril, and Echidna.
    • An "Isolation" test runs exclusively on the isolated container to ensure its security features (read-only filesystem, no network) are active.

Python Environment Management

  • asdf replaced with uv: The python version management has been migrated from asdf to uv. uv is a faster and more modern python package installer and resolver.

Other Changes

  • Minor updates to the README file.

mattaereal and others added 2 commits March 12, 2025 15:45
Accidentally pushed PR to main, will disable that after this update
@d4rm5 d4rm5 changed the title feat: restructure devcontainers and improve CI #15 feat: restructure devcontainers and improve CI #16 Aug 26, 2025
@d4rm5 d4rm5 requested a review from mattaereal August 26, 2025 14:25
@mattaereal mattaereal self-assigned this Aug 27, 2025
@mattaereal mattaereal merged commit 4225ee0 into main Aug 27, 2025
8 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