Skip to content

chore: new ci workflows#58

Merged
Danil42Russia merged 17 commits into
masterfrom
danil42russia/ci
Jun 28, 2026
Merged

chore: new ci workflows#58
Danil42Russia merged 17 commits into
masterfrom
danil42russia/ci

Conversation

@Danil42Russia

@Danil42Russia Danil42Russia commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Issue

Closes #

Checklist

Please ensure the following items are completed before requesting review:

  • Updated CHANGELOG.md with relevant changes
  • Documented the contribution in README.md
  • Added tests to demonstrate correct behavior (both positive and negative cases)
  • All tests pass successfully (yarn test)
  • Code passes linting checks (yarn lint)

Summary by CodeRabbit

  • New Features

    • Added automated linting, testing, security, and acceptance-check workflows.
    • Added a release pipeline to publish tagged versions and create GitHub releases.
  • Bug Fixes

    • Simplified CI by removing outdated or duplicate workflows.
  • Documentation

    • Minor updates to usage and contribution docs for clearer formatting.
  • Chores

    • Updated ignore rules and toolchain settings.
    • Refreshed formatting and package manager configuration.

@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Removes old qa.yaml, acceptance.yaml, publish.yml, reward*.yml workflows and replaces them with focused linter.yml, tests.yml, security.yml, acceptance.yml, and deploy.yml workflows targeting master. Yarn is upgraded from v3.6.1 to v4.17.0 with updated .yarnrc.yml, .gitignore, and package.json tooling config. Minor doc formatting edits applied.

Changes

CI/CD Workflow Overhaul

Layer / File(s) Summary
Yarn v4 upgrade and config
package.json, .yarnrc.yml, .gitignore, .prettierignore, tsconfig.json, template/.vscode/extensions.json
packageManager bumped to yarn@4.17.0; .yarnrc.yml gains approvedGitRepositories, enableScripts: false, and npmMinimalAgeGate: 7d; .gitignore simplifies .yarn/ ignores and adds yarn-error.log; .github added to .prettierignore; fmt/fmt:check scripts gain --cache; minor reformatting in tsconfig.json and template file.
Linter, tests, and security workflows
.github/workflows/linter.yml, .github/workflows/tests.yml, .github/workflows/security.yml
Adds three PR/push workflows targeting master: linter.yml checks Yarn deduplication, formatting, and linting; tests.yml builds and runs the test suite on ubuntu-latest Node 22; security.yml runs npm audit at moderate and high severity levels when dependency/config files change. All share concurrency cancellation and minimal permissions.
Acceptance test workflow
.github/workflows/acceptance.yml
Adds a two-job workflow: build packs create-ton into a tarball artifact; acceptance-tests matrix job (ubuntu/windows/macos × Node 20,24 × tact/func/tolk) downloads the artifact, installs the CLI globally, scaffolds a Counter project, builds it, and runs npm test.
Tag-based deploy workflow
.github/workflows/deploy.yml
Adds deploy.yml triggered on vX.Y.Z tags: publish job validates the tag matches package.json version, then builds and publishes to npm with OIDC; release job creates a GitHub Release with generated notes.
Doc formatting
README.md, contributing.md
Inserts a blank line in the README usage section; reformats a blockquote and step-5 note in contributing.md.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Poem

🐇 Old workflows fell like autumn leaves,
New pipelines bloom with master's breeze,
Yarn four hops in, all scripts in place,
Security checks run with rapid grace,
The rabbit stamps each tag with cheer — deploy! 🚀

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is just the template with placeholders and unchecked items, so it lacks the required concrete issue and completion details. Replace the placeholder issue with a real number and complete the checklist with the relevant changes and verification status.
Title check ❓ Inconclusive The title is related to the CI workflow changes, but it is too generic to clearly describe the main update. Use a more specific title like "chore: add new GitHub Actions CI workflows" or similar.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch danil42russia/ci

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Danil42Russia Danil42Russia merged commit 03eca7b into master Jun 28, 2026
3 of 4 checks passed
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