Skip to content

Fix template standardization issues and enforce AGENTS.md conventions#700

Open
sethoshi18 wants to merge 3 commits into
base:mainfrom
sethoshi18:jules-template-fixes-2215247443577211787
Open

Fix template standardization issues and enforce AGENTS.md conventions#700
sethoshi18 wants to merge 3 commits into
base:mainfrom
sethoshi18:jules-template-fixes-2215247443577211787

Conversation

@sethoshi18
Copy link
Copy Markdown

  • Updated gen-validation in Base Bridge Partner Template to use macro.
  • Refactored BasicScript.s.sol to correctly implement MultisigScript.
  • Enforced AGENTS.md status rules in READY TO SIGN for README files.

- Updated `gen-validation` in Base Bridge Partner Template to use macro.
- Refactored `BasicScript.s.sol` to correctly implement `MultisigScript`.
- Enforced `AGENTS.md` status rules in `READY TO SIGN` for README files.
@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 26, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/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 1
Sum 3

@@ -1,6 +1,6 @@
# Update Gas Limit, Elasticity & DA Footprint Gas Scalar in L1 `SystemConfig`

Status: TODO[READY TO SIGN|DONE]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes you made to the template READMEs are incorrect. Since it's a template, the status should not be set. Please revert all of these.

…dback

This reverts the changes to `setup-templates/*/README.md` to keep their original status placeholders, as these are base templates and should not have forced statuses.
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