Skip to content

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))

@vercel
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
@github-actions
Copy link

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

@codecov
Copy link

codecov bot commented Jul 12, 2025

Bundle Report

Bundle size has no change ✅

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