Skip to content

Comments

fix: incorrect title and status#602

Open
0x4tl4nt wants to merge 1 commit intobase:mainfrom
0x4tl4nt:0x4tl4nt-patch-1
Open

fix: incorrect title and status#602
0x4tl4nt wants to merge 1 commit intobase:mainfrom
0x4tl4nt:0x4tl4nt-patch-1

Conversation

@0x4tl4nt
Copy link

The title template-set-bridge-partner-threshold did not correspond to the purpose of the template, and the status was also incorrect. The issue was that the title and status were mistakenly copied from this file: template-pause-bridge-base

Changes:

  • Updated the README title to match the directory purpose: Set Bridge Partner Threshold.
  • Assigned README status Status: TEMPLATE, since the template-set-bridge-partner-threshold template is only a draft; Makefile copies it but does not run the signature directly. For comparison, the neighboring template-pause-bridge-base template is immediately used for signing and has the status Status: READY TO SIGN.

The title `template-set-bridge-partner-threshold` did not correspond to the purpose of the template, and the status was also incorrect. The issue was that the title and status were mistakenly copied from this file: [`template-pause-bridge-base`](https://github.com/0x4tl4nt/contract-deployments/blob/main/setup-templates/template-pause-bridge-base/README.md)

### Changes:
- Updated the `README` title to match the directory purpose: `Set Bridge Partner Threshold`.
- Assigned `README` status `Status: TEMPLATE`, since the `template-set-bridge-partner-threshold` template is only a draft; `Makefile` copies it but does not run the signature directly.
For comparison, the neighboring `template-pause-bridge-base` template is immediately used for signing and has the status `Status: READY TO SIGN`.
@cb-heimdall
Copy link
Collaborator

🟡 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 0
Sum 2

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.

2 participants