Skip to content
This repository was archived by the owner on May 21, 2026. It is now read-only.

Fix/release yml - #53

Open
Pederzh wants to merge 2 commits into
mainfrom
fix/release-yml
Open

Fix/release yml#53
Pederzh wants to merge 2 commits into
mainfrom
fix/release-yml

Conversation

@Pederzh

@Pederzh Pederzh commented Oct 5, 2025

Copy link
Copy Markdown
Contributor

Description

This PR improves the GitHub Actions workflows for better consistency, performance, and reliability:

Workflow Improvements:

  • Replace npm commands with pnpm for consistency with package.json (which specifies "packageManager": "pnpm@10.6.1")
  • Add pnpm store caching to speed up dependency installation across all workflow jobs
  • Use --frozen-lockfile for deterministic builds and faster CI runs
  • Fix changelog link to point to release page instead of broken comparison URL
  • Add proper pnpm setup across all workflow jobs for consistency

Testing Improvements:

  • Replace fallback echo with proper test:run command using vitest
  • Ensure test failures block releases as intended by using the correct test script
  • Use vitest run mode for CI environment (non-watch mode)

These changes align the CI/CD pipeline with the project's actual package manager configuration and improve build reliability and performance.

Fixes # (issue)

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

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have added tests that prove my fix is effective or that my feature works
  • The title of my pull request follows the conventional commits standard
  • Changes have been documented in the README/documentation (if applicable)

@Pederzh
Pederzh requested a review from tonxxd October 5, 2025 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant