From d2b49d5747431ff3f8d5cb31b153de9db78399d0 Mon Sep 17 00:00:00 2001 From: JosueBrenes Date: Mon, 23 Feb 2026 18:48:57 -0600 Subject: [PATCH 1/2] chore: update PR template and workflow configuration - Revised the pull request template for clarity and conciseness, changing section titles and removing unnecessary comments. - Standardized quotes in the GitHub Actions workflow configuration for Node.js setup. --- .github/PULL_REQUEST_TEMPLATE.md | 108 ++++--------------------------- .github/workflows/pr-checks.yml | 8 +-- 2 files changed, 17 insertions(+), 99 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index e89718a..de1b37b 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,99 +1,17 @@ -## Description +## What does this PR do? + - - -## Type of Change - - - -- [ ] 🐛 Bug fix (non-breaking change which fixes an issue) -- [ ] ✨ New feature (non-breaking change which adds functionality) -- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected) -- [ ] 📚 Documentation update -- [ ] 🎨 Style/formatting changes (no code change) -- [ ] ♻️ Code refactoring (no functional changes) -- [ ] ⚡ Performance improvement -- [ ] ✅ Tests (adding missing tests or correcting existing tests) -- [ ] 🔧 Build/config changes -- [ ] 🔒 Security fix - -## Related Issues - - - -Closes # -Fixes # -Relates to # - -## Changes Made - - - -- -- -- - -## Screenshots / Videos - - - -| Before | After | -| ------ | ----- | -| | | - -## Testing - - - -### Test Cases - -- [ ] Test case 1: -- [ ] Test case 2: -- [ ] Test case 3: - -### Manual Testing Steps - -1. -2. -3. +## Type of change +- [ ] Bug fix +- [ ] New feature +- [ ] Documentation +- [ ] Other (describe below) ## Checklist +- [ ] Code follows project style (lint passes) +- [ ] Changes have been tested locally - - -- [ ] My code follows the project's style guidelines -- [ ] I have performed a self-review of my own code -- [ ] I have commented my code, particularly in hard-to-understand areas -- [ ] I have made corresponding changes to the documentation -- [ ] My changes generate no new warnings -- [ ] I have added tests that prove my fix is effective or that my feature works -- [ ] New and existing unit tests pass locally with my changes -- [ ] Any dependent changes have been merged and published -- [ ] I have checked for breaking changes and documented them if applicable -- [ ] I have updated the CHANGELOG.md (if applicable) - -## Dependencies - - - -### Added - -- - -### Removed - -- - -### Updated - -- - -## Additional Notes - - - -## Reviewers - - - -/cc @ +## Video evidence (before / after) + +- **Before:** +- **After:** diff --git a/.github/workflows/pr-checks.yml b/.github/workflows/pr-checks.yml index ffdaa50..14adca4 100644 --- a/.github/workflows/pr-checks.yml +++ b/.github/workflows/pr-checks.yml @@ -15,8 +15,8 @@ jobs: - name: Setup Node.js uses: actions/setup-node@v4 with: - node-version: "20" - cache: "npm" + node-version: '20' + cache: 'npm' cache-dependency-path: package-lock.json - name: Install dependencies @@ -35,8 +35,8 @@ jobs: - name: Setup Node.js uses: actions/setup-node@v4 with: - node-version: "20" - cache: "npm" + node-version: '20' + cache: 'npm' cache-dependency-path: package-lock.json - name: Install dependencies From d30f9716ccbcdfed9c7d17932595ab443947ac21 Mon Sep 17 00:00:00 2001 From: JosueBrenes Date: Mon, 23 Feb 2026 18:52:43 -0600 Subject: [PATCH 2/2] chore: update Husky hooks for improved pre-commit and pre-push processes - Removed lint and build commands from pre-commit hook to streamline the commit process. - Added a new pre-push hook to enforce linting and building before pushing changes. --- .husky/pre-commit | 2 -- .husky/pre-push | 5 +++++ 2 files changed, 5 insertions(+), 2 deletions(-) create mode 100644 .husky/pre-push diff --git a/.husky/pre-commit b/.husky/pre-commit index 11a7e44..1bda9c1 100644 --- a/.husky/pre-commit +++ b/.husky/pre-commit @@ -2,5 +2,3 @@ . "$(dirname -- "$0")/_/husky.sh" npm run format -npm run lint -npm run build \ No newline at end of file diff --git a/.husky/pre-push b/.husky/pre-push new file mode 100644 index 0000000..0a7cec6 --- /dev/null +++ b/.husky/pre-push @@ -0,0 +1,5 @@ +#!/usr/bin/env sh +. "$(dirname -- "$0")/_/husky.sh" + +npm run lint +npm run build