Problem
The forge fmt --check step in our GitHub Actions workflow (.github/workflows/test.yml) is currently commented out. This was done as a temporary measure to unblock CI when there were formatting issues in the codebase.
Current State
- The formatting check step is commented out in the test workflow
- Formatting issues were fixed in commit df942e7 ("Fixed formatting")
- Without this check, inconsistently formatted code can be merged into the repository
Proposed Solution
Uncomment the forge fmt --check step in the CI workflow to:
- Enforce consistent Solidity code formatting standards
- Catch formatting issues early in the PR review process
- Maintain code readability and reduce formatting-related merge conflicts
Problem
The
forge fmt --checkstep in our GitHub Actions workflow (.github/workflows/test.yml) is currently commented out. This was done as a temporary measure to unblock CI when there were formatting issues in the codebase.Current State
Proposed Solution
Uncomment the
forge fmt --checkstep in the CI workflow to: