-
-
Notifications
You must be signed in to change notification settings - Fork 0
ci(*): update test-cross-platform.yml
#396
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Labels have been automatically applied based on the Conventional Commits specification.🏷️ |
Bundle ReportBundle size has no change ✅ |
Codecov ReportAll modified and coverable lines are covered by tests ✅
@@ Coverage Diff @@
## main #396 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 11 11
Lines 422 422
=========================================
Hits 422 422 Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enhances the cross-platform CI workflow by simplifying script steps, expanding test coverage matrices, and removing redundant matrix parameters.
- Consolidates package build commands into a single line
- Adds
fail-fast: false
and broadens OS/node version matrices - Drops unnecessary
ubuntu-image
fields and simplifies Docker runs
Comments suppressed due to low confidence (1)
.github/workflows/test-cross-platform.yml:45
- Chaining commands with
&&
may fail on Windows PowerShell 5.1 (default shell on Windows runners). Consider specifyingshell: bash
for this step or splitting into separate commands to ensure compatibility across all platforms.
run: mkdir packs && npm pack --workspace packages --pack-destination packs
…ch`) (#399) ## Release Information: `v1.3.4` New release of `lumirlumir/npm-clang-format-node` has arrived! 🎉 This PR bumps the package versions from `v1.3.3` to `v1.3.4` (`patch`). See [Actions](https://github.com/lumirlumir/npm-clang-format-node/actions/runs/16236437387) for more details. | Info | Value | | ----------- | -------------------------- | | Repository | `lumirlumir/npm-clang-format-node` | | SEMVER | `patch` | | Pre ID | `canary` | | Short SHA | 5862509 | | Old Version | `v1.3.3` | | New Version | `v1.3.4` | <!-- Release notes generated using configuration in .github/release.yml at main --> ## What's Changed ### 🐛 Bug Fixes * fix(*): update `postinstall` script to remove unnecessary build check by @lumirlumir in #398 ### 🧰 Chores * chore(*): update configuration files to use `.cjs` or `.mjs` extensions by @lumirlumir in #298 * chore(sync-server): update lint configs and bump package versions by @lumirlumir in #310 * chore(sync-server): update `lint-staged` and scripts by @lumirlumir in #314 * chore(sync-server): update `dependabot.yml` and `.gitignore` by @lumirlumir in #326 * chore(sync-server): remove `is-interactive` dependency and update Codecov configuration by @lumirlumir in #336 * chore(sync-server): update `dependabot.yml` by @lumirlumir in #351 * chore(*): run `npm dedupe` for cleanup by @lumirlumir in #362 * chore(*): remove `bananass-utils-vitepress` dependency by @lumirlumir in #377 * chore(*): move `FUNDING.yml` files to `.github` repository by @lumirlumir in #389 * chore(*): replace `shx` with native Node.js file operations in scripts by @lumirlumir in #397 ### 🔄 Continuous Integrations * ci(*): update actions to use docker native action by @lumirlumir in #393 * ci(*): update `test-cross-platform.yml` by @lumirlumir in #396 ### 🎨 Styles * style(*): clean up comments and remove unnecessary whitespace in multiple files by @lumirlumir in #380 ### 🧪 Tests * test(*): simplify describe blocks in test files by @lumirlumir in #331 ### ⬆️ Dependency Updates * chore(deps-dev): bump @types/node from 22.13.17 to 22.14.0 by @dependabot[bot] in #299 * chore(deps-dev): bump eslint-config-bananass from 0.0.6 to 0.0.7 in the bananass group across 1 directory by @dependabot[bot] in #300 * chore(deps-dev): bump eslint from 9.23.0 to 9.24.0 by @dependabot[bot] in #301 * chore(deps-dev): bump typescript from 5.8.2 to 5.8.3 by @dependabot[bot] in #302 * chore(deps-dev): bump vitepress-plugin-group-icons from 1.3.8 to 1.4.1 by @dependabot[bot] in #303 * chore(deps-dev): bump lerna from 8.2.1 to 8.2.2 by @dependabot[bot] in #305 * chore(deps-dev): bump @types/node from 22.14.0 to 22.14.1 by @dependabot[bot] in #306 * chore(deps-dev): bump textlint-rule-allowed-uris from 1.0.9 to 1.1.0 by @dependabot[bot] in #308 * chore(deps-dev): bump vitepress-plugin-group-icons from 1.4.1 to 1.5.1 by @dependabot[bot] in #307 * chore(deps-dev): bump lint-staged from 15.5.0 to 15.5.1 by @dependabot[bot] in #311 * chore(deps-dev): bump vitepress-plugin-group-icons from 1.5.1 to 1.5.2 by @dependabot[bot] in #312 * chore(deps-dev): bump eslint from 9.24.0 to 9.25.0 by @dependabot[bot] in #315 * chore(deps-dev): bump eslint from 9.25.0 to 9.25.1 by @dependabot[bot] in #316 * chore(deps-dev): bump @types/node from 22.14.1 to 22.15.0 by @dependabot[bot] in #318 * chore(deps-dev): bump the bananass group across 1 directory with 2 updates by @dependabot[bot] in #317 * chore(deps-dev): bump @types/node from 22.15.0 to 22.15.2 by @dependabot[bot] in #320 * chore(deps-dev): bump the bananass group across 1 directory with 2 updates by @dependabot[bot] in #319 * chore(deps-dev): bump @types/node from 22.15.2 to 22.15.3 by @dependabot[bot] in #321 * chore(deps-dev): bump the babel group across 1 directory with 3 updates by @dependabot[bot] in #322 * chore(deps-dev): bump textlint from 14.6.0 to 14.7.1 by @dependabot[bot] in #323 * chore(deps-dev): bump eslint from 9.25.1 to 9.26.0 by @dependabot[bot] in #324 * chore(deps-dev): bump @types/node from 22.15.3 to 22.15.9 by @dependabot[bot] in #325 * chore(deps-dev): bump the babel group across 1 directory with 2 updates by @dependabot[bot] in #327 * chore(deps-dev): bump lint-staged from 15.5.1 to 15.5.2 by @dependabot[bot] in #328 * chore(deps-dev): bump @types/node from 22.15.9 to 22.15.15 by @dependabot[bot] in #330 * chore(deps-dev): bump the bananass group across 1 directory with 2 updates by @dependabot[bot] in #332 * chore(deps-dev): bump @types/node from 22.15.16 to 22.15.17 by @dependabot[bot] in #333 * chore(deps-dev): bump lint-staged from 15.5.2 to 16.0.0 by @dependabot[bot] in #334 * chore(deps-dev): bump the bananass group across 2 directories with 3 updates by @dependabot[bot] in #335 * chore(deps): bump undici from 6.21.1 to 6.21.3 in the npm_and_yarn group across 1 directory by @dependabot[bot] in #340 * chore(deps-dev): bump vitepress-plugin-group-icons from 1.5.2 to 1.5.5 by @dependabot[bot] in #341 * chore(deps-dev): bump @types/node from 22.15.17 to 22.15.18 by @dependabot[bot] in #338 * chore(deps-dev): bump eslint from 9.26.0 to 9.27.0 by @dependabot[bot] in #345 * chore(deps-dev): bump markdownlint-cli from 0.44.0 to 0.45.0 by @dependabot[bot] in #344 * chore(deps-dev): bump @codecov/vite-plugin from 1.9.0 to 1.9.1 by @dependabot[bot] in #346 * chore(deps-dev): bump the bananass group across 2 directories with 3 updates by @dependabot[bot] in #348 * chore(deps-dev): bump @types/node from 22.15.18 to 22.15.21 by @dependabot[bot] in #350 * chore(deps-dev): bump textlint from 14.7.1 to 14.7.2 by @dependabot[bot] in #349 * chore(deps-dev): bump @types/node from 22.15.21 to 22.15.24 by @dependabot[bot] in #354 * chore(deps-dev): bump @babel/core from 7.27.1 to 7.27.3 in the babel group across 1 directory by @dependabot[bot] in #352 * chore(deps-dev): bump @types/node from 22.15.24 to 22.15.26 by @dependabot[bot] in #355 * chore(deps-dev): bump lint-staged from 16.0.0 to 16.1.0 by @dependabot[bot] in #356 * chore(deps-dev): bump eslint from 9.27.0 to 9.28.0 by @dependabot[bot] in #358 * chore(deps-dev): bump @babel/core from 7.27.3 to 7.27.4 in the babel group across 1 directory by @dependabot[bot] in #357 * chore(deps-dev): bump @types/node from 22.15.26 to 22.15.29 by @dependabot[bot] in #359 * chore(deps-dev): bump vitepress-plugin-group-icons from 1.5.5 to 1.6.0 by @dependabot[bot] in #360 * chore(deps-dev): bump the bananass group across 2 directories with 3 updates by @dependabot[bot] in #364 * chore(deps-dev): bump textlint-rule-allowed-uris from 1.1.0 to 1.1.1 by @dependabot[bot] in #366 * chore(deps-dev): bump @types/node from 22.15.29 to 24.0.0 by @dependabot[bot] in #367 * chore(deps-dev): bump @types/node from 24.0.0 to 24.0.1 by @dependabot[bot] in #368 * chore(deps-dev): bump eslint from 9.28.0 to 9.29.0 by @dependabot[bot] in #370 * chore(deps-dev): bump lint-staged from 16.1.0 to 16.1.2 by @dependabot[bot] in #369 * chore(deps-dev): bump @types/node from 24.0.1 to 24.0.3 by @dependabot[bot] in #371 * chore(deps-dev): bump concurrently from 9.1.2 to 9.2.0 by @dependabot[bot] in #372 * chore(deps-dev): bump prettier from 3.5.3 to 3.6.0 by @dependabot[bot] in #373 * chore(deps-dev): bump @babel/core from 7.27.4 to 7.27.7 in the babel group across 1 directory by @dependabot[bot] in #376 * chore(deps-dev): bump prettier from 3.6.0 to 3.6.2 by @dependabot[bot] in #378 * chore(deps-dev): bump eslint from 9.29.0 to 9.30.0 by @dependabot[bot] in #382 * chore(deps-dev): bump lerna from 8.2.2 to 8.2.3 by @dependabot[bot] in #381 * chore(deps-dev): bump vitepress-plugin-group-icons from 1.6.0 to 1.6.1 by @dependabot[bot] in #383 * chore(deps-dev): bump @types/node from 24.0.3 to 24.0.8 by @dependabot[bot] in #385 * chore(deps-dev): bump @types/node from 24.0.8 to 24.0.10 by @dependabot[bot] in #387 * chore(deps-dev): bump eslint from 9.30.0 to 9.30.1 by @dependabot[bot] in #386 * chore(deps-dev): bump the babel group across 1 directory with 3 updates by @dependabot[bot] in #388 * chore(deps-dev): bump the bananass group across 1 directory with 2 updates by @dependabot[bot] in #391 **Full Changelog**: v1.3.3...v1.3.4
This pull request updates the
.github/workflows/test-cross-platform.yml
file to improve workflow efficiency and expand platform and node version coverage. Key changes include simplifying commands, adding new matrix configurations, and removing redundant parameters.Workflow efficiency improvements:
Build packages
step by combining commands into a single line using&&
. ([.github/workflows/test-cross-platform.ymlL45-R45](https://github.com/lumirlumir/npm-clang-format-node/pull/396/files#diff-290907d9a4bcf13181105b134bd1645da5a3ef27c54f148ad5823e6b77e88426L45-R45)
)ubuntu-image
parameters in thedocker-images
matrix and updated theDocker run background
step to use a simplified image reference. ([[1]](https://github.com/lumirlumir/npm-clang-format-node/pull/396/files#diff-290907d9a4bcf13181105b134bd1645da5a3ef27c54f148ad5823e6b77e88426R133-L144)
,[[2]](https://github.com/lumirlumir/npm-clang-format-node/pull/396/files#diff-290907d9a4bcf13181105b134bd1645da5a3ef27c54f148ad5823e6b77e88426L159-R157)
)Expanded coverage:
fail-fast: false
to bothgithub-actions-runner-images
anddocker-images
strategies to ensure all matrix combinations are tested even if one fails. ([[1]](https://github.com/lumirlumir/npm-clang-format-node/pull/396/files#diff-290907d9a4bcf13181105b134bd1645da5a3ef27c54f148ad5823e6b77e88426R55-R66)
,[[2]](https://github.com/lumirlumir/npm-clang-format-node/pull/396/files#diff-290907d9a4bcf13181105b134bd1645da5a3ef27c54f148ad5823e6b77e88426R133-L144)
)runner-image
matrix to includewindows-2025
and added Node.js version24
to thenode-version
matrix. ([.github/workflows/test-cross-platform.ymlR55-R66](https://github.com/lumirlumir/npm-clang-format-node/pull/396/files#diff-290907d9a4bcf13181105b134bd1645da5a3ef27c54f148ad5823e6b77e88426R55-R66)
)