Skip to content

feat: add CI/CD workflows and smoke tests#1

Merged
bntvllnt merged 3 commits intomainfrom
feat/ci-workflows
Mar 10, 2026
Merged

feat: add CI/CD workflows and smoke tests#1
bntvllnt merged 3 commits intomainfrom
feat/ci-workflows

Conversation

@bntvllnt
Copy link
Collaborator

Summary

  • ci.yml: PR quality gate — runs smoke tests on Node 22
  • publish.yml: Canary publish on push to main + release via workflow_dispatch (matches @vllnt/eslint-config pattern)
  • Smoke tests: Validate JSON syntax, compilerOptions presence, extends chains, and package.json files array
  • Uses npm OIDC provenance — no NPM_TOKEN secret needed

Test plan

  • npm test passes locally (3/3 tests green)
  • CI workflow runs on this PR
  • Publish workflow structure matches @vllnt/eslint-config

- ci.yml: PR quality gate (validate JSON configs on Node 22)
- publish.yml: canary on push to main + release via workflow_dispatch
- Smoke tests validate JSON syntax, extends chains, and package.json files array
- Uses npm OIDC provenance (matches @vllnt/eslint-config pattern)
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

- Use annotated tags (git tag -a) instead of lightweight
- Use --notes-file for GitHub Release (avoids shell interpolation issues)
- Simplify release title to version-only format
@bntvllnt bntvllnt merged commit bd3fb92 into main Mar 10, 2026
1 check passed
@bntvllnt bntvllnt deleted the feat/ci-workflows branch March 10, 2026 17: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.

1 participant