-
-
Notifications
You must be signed in to change notification settings - Fork 0
chore(*): replace shx
with native Node.js file operations in scripts
#397
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
chore(*): replace shx
with native Node.js file operations in scripts
#397
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Labels have been automatically applied based on the Conventional Commits specification.🏷️ |
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 replaces the shx
-based file copy in the example script with a native Node.js approach and removes the now-unneeded shx
dependency.
- Removes the
shx
dependency from the rootpackage.json
. - Updates the example
add-a-space-to-line-9-of-main-c-file
script to usefs.copyFileSync
vianode -e
. - Simplifies script usage by relying on Node’s built-in file system API.
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
package.json | Removed the shx dependency from devDependencies |
examples/git-clang-format/package.json | Swapped out the shx command for a native fs.copyFileSync call |
Comments suppressed due to low confidence (1)
examples/git-clang-format/package.json:6
- [nitpick] The script name implies adding a space at line 9, but the implementation copies the entire file. Consider renaming the script or updating its description to accurately reflect its behavior.
"add-a-space-to-line-9-of-main-c-file": "node -e \"require('fs').copyFileSync('src/main_overwrite.txt','src/main.c')\"",
Bundle ReportChanges will increase total bundle size by 12 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: website-esmAssets Changed:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
@@ Coverage Diff @@
## main #397 +/- ##
=========================================
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:
|
…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
examples/git-clang-format/package.json
file to replace the use ofshx
with a native Node.js solution and removes theshx
dependency from the project'spackage.json
.Dependency updates:
package.json
: Removed theshx
dependency, as it is no longer needed after switching to a native Node.js approach.Script updates:
examples/git-clang-format/package.json
: Replaced theshx
command in theadd-a-space-to-line-9-of-main-c-file
script with a Node.jsfs.copyFileSync
operation for copying files.