Skip to content

Update .pre-commit-config.yaml#1435

Open
LeanAira10 wants to merge 1 commit intodymensionxyz:mainfrom
LeanAira10:patch-1
Open

Update .pre-commit-config.yaml#1435
LeanAira10 wants to merge 1 commit intodymensionxyz:mainfrom
LeanAira10:patch-1

Conversation

@LeanAira10
Copy link

@LeanAira10 LeanAira10 commented Jan 25, 2026

Optimizations:

Security: Added detect-private-key to block accidental commits of RSA/SSH keys and sensitive .env data.

Integrity: Enabled check-merge-conflict to ensure no git merge markers break the CI pipeline.

Maintenance: Added go-mod-tidy to enforce dependency consistency before commits.

Reliability: Included a basic go-build check to catch syntax errors that linters might miss.

Performance: Switched to language: system to bypass virtual environment creation and leverage host machine tools for faster execution.

PR Standards

Opening a pull request should be able to meet the following requirements


For Author:

  • Targeted PR against correct branch
  • Included the correct type prefix in the PR title
  • Linked to Github issue with discussion and accepted design
  • Targets only one github issue
  • Wrote unit and integration tests
  • All CI checks have passed
  • Added relevant godoc comments

For Reviewer:

  • Confirmed the correct type prefix in the PR title
  • Reviewers assigned
  • Confirmed all author checklist items have been addressed

After reviewer approval:

  • In case PR targets main branch, PR should be squashed and merged.
  • In case PR targets a release branch, PR should be rebased.

Optimizations:

Security: Added detect-private-key to block accidental commits of RSA/SSH keys and sensitive .env data.

Integrity: Enabled check-merge-conflict to ensure no git merge markers break the CI pipeline.

Maintenance: Added go-mod-tidy to enforce dependency consistency before commits.

Reliability: Included a basic go-build check to catch syntax errors that linters might miss.

Performance: Switched to language: system to bypass virtual environment creation and leverage host machine tools for faster execution.
@LeanAira10 LeanAira10 requested a review from a team as a code owner January 25, 2026 19:18
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.

1 participant