Skip to content

Add CI workflow for automated testing and validation - #9

Merged
IdoS350 merged 6 commits into
mainfrom
claude/add-github-actions-as8Q1
May 8, 2026
Merged

IdoS350 merged 6 commits into
mainfrom
claude/add-github-actions-as8Q1

Conversation

@IdoS350

@IdoS350 IdoS350 commented May 8, 2026

Copy link
Copy Markdown
Owner

Summary

This PR introduces a GitHub Actions CI workflow to automate code quality checks and testing on every push to main and pull request.

Key Changes

  • Added .github/workflows/ci.yml with automated CI pipeline that runs on:
    • Push events to the main branch
    • Pull requests targeting the main branch

Workflow Steps

The CI pipeline performs the following checks in sequence:

  • Typecheck: Validates TypeScript types
  • Format: Ensures code formatting compliance
  • Test: Runs the test suite
  • Build: Verifies the project builds successfully

Implementation Details

  • Uses Node.js 22 with pnpm package manager
  • Leverages pnpm caching for faster dependency installation
  • Installs dependencies with --frozen-lockfile to ensure reproducible builds
  • All steps must pass for the workflow to succeed

https://claude.ai/code/session_01Q8Nea1J7ToUHtVVAjwz7QS

claude and others added 6 commits May 7, 2026 15:51
Runs typecheck, format check, tests, and build on push/PR to main.

https://claude.ai/code/session_01Q8Nea1J7ToUHtVVAjwz7QS
pnpm/action-setup requires either a version key in the workflow or a
packageManager field in package.json; using the latter is the canonical
approach.

https://claude.ai/code/session_01Q8Nea1J7ToUHtVVAjwz7QS
- Set FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true to silence Node.js 20
  deprecation warnings from actions/checkout, actions/setup-node, and
  pnpm/action-setup
- Add pnpm.onlyBuiltDependencies to approve build scripts for esbuild,
  @vscode/vsce-sign, and keytar, silencing the ignored-build-scripts warning

https://claude.ai/code/session_01Q8Nea1J7ToUHtVVAjwz7QS
actions/checkout, actions/setup-node, and pnpm/action-setup all moved to
Node 24 in v5, so bumping to v6 is the proper fix rather than the
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24 env var workaround.

https://claude.ai/code/session_01Q8Nea1J7ToUHtVVAjwz7QS
@IdoS350
IdoS350 merged commit 0b332ee into main May 8, 2026
1 check passed
@IdoS350
IdoS350 deleted the claude/add-github-actions-as8Q1 branch May 8, 2026 12:37
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