build(*): add cp.mjs script for file copying and update build scripts
#400
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Labels have been automatically applied based on the Conventional Commits specification.🏷️ |
cp.mjs script for file copying and update build scripts
There was a problem hiding this comment.
Pull Request Overview
This PR adds a reusable Node-based file copy utility and standardizes copy commands across package build scripts, updates CI/docker workflows for apt-get, and ensures the new script is synced via GitHub configurations.
- Introduces
scripts/cp.mjsfor recursive, forced file copying. - Updates package
buildscripts to usenode ../../scripts/cp.mjsinstead ofshx cp. - Replaces
aptwithapt-getin CI workflows and documentation; syncs the new script in GitHub sync client.
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| website/docs/further-reading/build-process.md | Swapped apt -> apt-get in build instructions |
| scripts/cp.mjs | Added new reusable copy script |
| packages/clang-format-node/package.json | Updated build script to use cp.mjs |
| packages/clang-format-git/package.json | Updated build script to use cp.mjs |
| packages/clang-format-git-python/package.json | Updated build script to use cp.mjs |
| .github/workflows/test-cross-platform.yml | Swapped apt -> apt-get in Docker setup steps |
| .github/sync-client.yml | Added scripts/cp.mjs to sync client configuration |
Comments suppressed due to low confidence (2)
scripts/cp.mjs:1
- There are no tests covering this new copy utility. Adding unit or integration tests will help ensure it behaves as expected under different scenarios.
/**
scripts/cp.mjs:1
- [nitpick] Add a note in the project README or a top-level docs file to explain the purpose and usage of
cp.mjs, so consumers know when and how to use it.
/**
Bundle ReportChanges will increase total bundle size by 31 bytes (0.01%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: website-esmAssets Changed:
Files in
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
@@ Coverage Diff @@
## main #400 +/- ##
=========================================
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:
|
lumirlumir
added a commit
that referenced
this pull request
Jul 14, 2025
…ch`) (#404) ## Release Information: `v1.3.5` New release of `lumirlumir/npm-clang-format-node` has arrived! 🎉 This PR bumps the package versions from `v1.3.4` to `v1.3.5` (`patch`). See [Actions](https://github.com/lumirlumir/npm-clang-format-node/actions/runs/16264909140) for more details. | Info | Value | | ----------- | -------------------------- | | Repository | `lumirlumir/npm-clang-format-node` | | SEMVER | `patch` | | Pre ID | `canary` | | Short SHA | df49101 | | Old Version | `v1.3.4` | | New Version | `v1.3.5` | <!-- Release notes generated using configuration in .github/release.yml at main --> ## What's Changed ### 🐛 Bug Fixes * fix(*): drop `shx` and replace it with Node.js native script by @lumirlumir in #402 ### 🛠️ Builds * build(*): add `cp.mjs` script for file copying and update build scripts by @lumirlumir in #400 ### ♻️ Code Refactoring * refactor(*): simplify paths to the binary files by @lumirlumir in #401 ### ⬆️ Dependency Updates * chore(deps-dev): bump eslint from 9.30.1 to 9.31.0 by @dependabot[bot] in #403 **Full Changelog**: v1.3.4...v1.3.5
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces updates to streamline file copying processes across multiple packages, refactor scripts for consistency, and improve documentation. The most significant changes involve the addition of a reusable file copy script (
cp.mjs), updates to package build commands to use this script, and the replacement ofaptcommands withapt-getfor better compatibility.File Copying Enhancements:
scripts/cp.mjsto facilitate file copying with support for recursive and forced operations. This script is now reusable across packages.buildscripts inpackages/clang-format-git-python/package.json,packages/clang-format-git/package.json, andpackages/clang-format-node/package.jsonto replaceshx cpcommands withnode ../../scripts/cp.mjs, enhancing consistency and maintainability. [1] [2] [3]Configuration Updates:
cp.mjsto.github/sync-client.ymlfor syncing scripts across repositories.Compatibility Improvements:
aptcommands withapt-getin.github/workflows/test-cross-platform.ymlandwebsite/docs/further-reading/build-process.mdfor improved compatibility and standardization. [1] [2]