Skip to content

ci: validate active/evm common scripts instead of legacy templates - #775

Merged
0xth4nh merged 1 commit into
mainfrom
mux/ci-validate-common
Jul 29, 2026
Merged

ci: validate active/evm common scripts instead of legacy templates#775
0xth4nh merged 1 commit into
mainfrom
mux/ci-validate-common

Conversation

@0xth4nh

@0xth4nh 0xth4nh commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

What changed?

Replaces the setup-templates validation matrix with a single job that validates the shared active/evm project:

  • Removes .github/workflows/validate-templates.yml (built each old standalone setup-templates/template-* project).
  • Adds .github/workflows/validate-common-scripts.yml: installs deps into active/evm (make deps PROJECT_DIR=…, base-contracts pinned via env) and runs forge fmt --check script/ + forge build over script/common.

Stacked on #773 (foundation).

Why?

The old workflow tested the legacy template projects, which the shared active/evm + script/common model replaces. This gives CI coverage for the new common scripts (fmt + compile).

How to test?

make deps PROJECT_DIR="$PWD/active/evm"
cd active/evm
mise exec -- forge fmt --check script/
mise exec -- forge build

All pass locally against base-contracts v8.2.1.

Base automatically changed from mux/active-evm-foundation to main July 29, 2026 22:01
@cb-heimdall

cb-heimdall commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 2
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 2
2
1 if commit is unverified 0
Sum 2

@base base deleted a comment from CatfishbillyIII Jul 29, 2026
@0xth4nh
0xth4nh force-pushed the mux/ci-validate-common branch from ce69c3a to 5367a22 Compare July 29, 2026 22:16
Comment thread .github/workflows/validate-common-scripts.yml Fixed
Replaces the setup-templates matrix (which built each old standalone
template project) with a single job that installs deps into the shared
active/evm Foundry project and runs forge fmt --check + forge build over
script/common.

Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
@0xth4nh
0xth4nh force-pushed the mux/ci-validate-common branch from 5367a22 to 035093b Compare July 29, 2026 22:19
@0xth4nh
0xth4nh merged commit 00c1200 into main Jul 29, 2026
5 checks passed
@0xth4nh
0xth4nh deleted the mux/ci-validate-common branch July 29, 2026 22:55
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.

5 participants