Closed fengmk2 closed 3 months ago
The recent updates involve refining the GitHub Actions workflows by removing the workflow_dispatch
trigger from two configuration files, enhancing the structure of test scripts in package.json
, and adding a new gzip compression test in Node.js. These changes streamline the development process and expand testing capabilities.
File Path | Change Summary |
---|---|
.github/workflows/... |
Removed workflow_dispatch trigger from nodejs.yml and release.yml , also removed checkTest from release.yml . |
package.json |
Updated test scripts: split ts-test into test:ts and test:js , adjusted ci script. |
test/gzip/file_stream.test.js |
Added test for compressing a file into a Buffer using gzip. |
🐇💻✨ Oh hoppy day for code and play, With tests that split and workflows sway. Gzip streams compress, oh what a fest, A rabbit's joy in code expressed! 🎉📜🐾
.github/workflows/release.yml (1)
`4-13`: > :memo: **NOTE** > This review was outside the diff hunks and was mapped to the diff hunk with the greatest overlap. Original lines [1-10] Verify the impact of removing the `workflow_dispatch` trigger and the `checkTest` parameter. Ensure that the removal of these features aligns with your project's workflow requirements and does not inadvertently reduce necessary checks or flexibility in the release process.package.json (1)
`8-14`: LGTM! The reorganization of test scripts enhances modularity and clarity. Ensure that the CI environment is correctly configured to handle these changes without issues.test/gzip/file_stream.test.js (1)
`37-48`: LGTM! The new test case correctly implements the functionality for compressing a file into a Buffer using gzip. Consider adding more detailed assertions to verify the contents of the compressed file to ensure that the compression was performed correctly.
closes https://github.com/node-modules/compressing/issues/103
Summary by CodeRabbit
Chores
workflow_dispatch
trigger from Node.js workflow configurations.Tests
package.json
for better clarity and execution.