Skip to content

chore: add PR template#221

Open
init4samwise wants to merge 2 commits intomainfrom
samwise/add-pr-template
Open

chore: add PR template#221
init4samwise wants to merge 2 commits intomainfrom
samwise/add-pr-template

Conversation

@init4samwise
Copy link
Contributor

Description

Adds a standardized PR template for the builder repo with:

  • Linear ticket linking (ENG-XXXX format)
  • Conventional commit format reference
  • Builder-specific checklist (block construction, MEV, performance)
  • Standard Rust code quality checks

Related Issue

Closes ENG-1818

Changes Made

  • Added .github/PULL_REQUEST_TEMPLATE.md

This is part of a broader effort to standardize PR templates across all Signet repositories.

Closes ENG-1818
Comment on lines 60 to 61
- [ ] `cargo fmt --check` passes
- [ ] `cargo clippy` passes without warnings
Copy link
Contributor

Choose a reason for hiding this comment

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

@init4samwise Specify the make targets for these instead.

@init4samwise
Copy link
Contributor Author

Updated to use make fmt and make clippy targets as requested. Ready for re-review!

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