Skip to content

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

Merged
merged 2 commits into from
Jul 12, 2025

Conversation

lumirlumir
Copy link
Owner

@lumirlumir lumirlumir commented Jul 12, 2025

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:

  • Simplified the 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))
  • Removed redundant ubuntu-image parameters in the docker-images matrix and updated the Docker 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:

  • Added fail-fast: false to both github-actions-runner-images and docker-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))
  • Expanded runner-image matrix to include windows-2025 and added Node.js version 24 to the node-version matrix. ([.github/workflows/test-cross-platform.ymlR55-R66](https://github.com/lumirlumir/npm-clang-format-node/pull/396/files#diff-290907d9a4bcf13181105b134bd1645da5a3ef27c54f148ad5823e6b77e88426R55-R66))

Copy link

vercel bot commented Jul 12, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
npm-clang-format-node-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 12, 2025 3:47am

@github-actions github-actions bot added 🏷️ scope: * Auto-generated label based on Conventional Commits specification for GitHub release notes 🏷️ type: ci Auto-generated label based on Conventional Commits specification for GitHub release notes labels Jul 12, 2025
Copy link

Labels have been automatically applied based on the Conventional Commits specification.🏷️

Copy link

codecov bot commented Jul 12, 2025

Bundle Report

Bundle size has no change ✅

Copy link

codecov bot commented Jul 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (c984171) to head (b8d0255).
Report is 1 commits behind head on main.

@@            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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c984171...b8d0255. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lumirlumir lumirlumir marked this pull request as ready for review July 12, 2025 04:32
@Copilot Copilot AI review requested due to automatic review settings July 12, 2025 04:32
Copy link
Contributor

@Copilot Copilot AI left a 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 specifying shell: 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

@lumirlumir lumirlumir merged commit 69a35ee into main Jul 12, 2025
54 checks passed
@lumirlumir lumirlumir deleted the ci-update-test-cross-platform-yml branch July 12, 2025 04:33
lumirlumir added a commit that referenced this pull request Jul 12, 2025
…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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ scope: * Auto-generated label based on Conventional Commits specification for GitHub release notes 🏷️ type: ci Auto-generated label based on Conventional Commits specification for GitHub release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant