Skip to content

ci: add pull request validation workflow#12

Merged
IISweetHeartII merged 1 commit intomainfrom
ci/add-pr-validation
Apr 17, 2026
Merged

ci: add pull request validation workflow#12
IISweetHeartII merged 1 commit intomainfrom
ci/add-pr-validation

Conversation

@IISweetHeartII
Copy link
Copy Markdown
Contributor

Adds GitHub Actions CI for lint, type-check, test, and build so SDK PRs like #11 can be validated.

Copy link
Copy Markdown
Contributor Author

@IISweetHeartII IISweetHeartII left a comment

Choose a reason for hiding this comment

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

Verdict: APPROVE-worthy, self-authored so leaving a comment-only review.

  • Workflow covers lint, type-check, test, and build on supported Node versions, which is exactly the missing guardrail for this repo.
  • passWithNoTests is a sensible bootstrap choice so CI does not fail before test files land.
  • No blocking issues found.

@IISweetHeartII IISweetHeartII merged commit 9999bd8 into main Apr 17, 2026
2 checks passed
@IISweetHeartII IISweetHeartII deleted the ci/add-pr-validation branch April 17, 2026 23:16
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