From d71fd73acccec5180c147ae63e99f63dc29c331b Mon Sep 17 00:00:00 2001 From: Les Dyman Date: Mon, 9 Feb 2026 12:14:18 +0100 Subject: [PATCH 1/2] add copy file solution --- .github/workflows/test.yml-template | 23 ++++++++++++++++++ package-lock.json | 9 ++++---- package.json | 2 +- src/app.js | 36 +++++++++++++++++++++++++++++ 4 files changed, 65 insertions(+), 5 deletions(-) create mode 100644 .github/workflows/test.yml-template diff --git a/.github/workflows/test.yml-template b/.github/workflows/test.yml-template new file mode 100644 index 0000000..bb13dfc --- /dev/null +++ b/.github/workflows/test.yml-template @@ -0,0 +1,23 @@ +name: Test + +on: + pull_request: + branches: [ master ] + +jobs: + build: + + runs-on: ubuntu-latest + + strategy: + matrix: + node-version: [20.x] + + steps: + - uses: actions/checkout@v2 + - name: Use Node.js ${{ matrix.node-version }} + uses: actions/setup-node@v1 + with: + node-version: ${{ matrix.node-version }} + - run: npm install + - run: npm test diff --git a/package-lock.json b/package-lock.json index 10f9f16..b7c274c 100644 --- a/package-lock.json +++ b/package-lock.json @@ -12,7 +12,7 @@ "devDependencies": { "@faker-js/faker": "^8.4.1", "@mate-academy/eslint-config": "latest", - "@mate-academy/scripts": "^1.8.6", + "@mate-academy/scripts": "^2.1.3", "eslint": "^8.57.0", "eslint-plugin-jest": "^28.6.0", "eslint-plugin-node": "^11.1.0", @@ -1484,10 +1484,11 @@ } }, "node_modules/@mate-academy/scripts": { - "version": "1.8.6", - "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-1.8.6.tgz", - "integrity": "sha512-b4om/whj4G9emyi84ORE3FRZzCRwRIesr8tJHXa8EvJdOaAPDpzcJ8A0sFfMsWH9NUOVmOwkBtOXDu5eZZ00Ig==", + "version": "2.1.3", + "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-2.1.3.tgz", + "integrity": "sha512-a07wHTj/1QUK2Aac5zHad+sGw4rIvcNl5lJmJpAD7OxeSbnCdyI6RXUHwXhjF5MaVo9YHrJ0xVahyERS2IIyBQ==", "dev": true, + "license": "MIT", "dependencies": { "@octokit/rest": "^17.11.2", "@types/get-port": "^4.2.0", diff --git a/package.json b/package.json index 4e4705d..1640507 100644 --- a/package.json +++ b/package.json @@ -18,7 +18,7 @@ "devDependencies": { "@faker-js/faker": "^8.4.1", "@mate-academy/eslint-config": "latest", - "@mate-academy/scripts": "^1.8.6", + "@mate-academy/scripts": "^2.1.3", "eslint": "^8.57.0", "eslint-plugin-jest": "^28.6.0", "eslint-plugin-node": "^11.1.0", diff --git a/src/app.js b/src/app.js index ad9a93a..93b6a76 100644 --- a/src/app.js +++ b/src/app.js @@ -1 +1,37 @@ 'use strict'; + +const fs = require('fs'); +const path = require('path'); + +/** + * @param {string} from + * @param {string} to + */ + +function copyFile(from, to) { + const resolveFrom = path.resolve(from); + const resolveTo = path.resolve(to); + + if (resolveFrom === resolveTo) { + return; + } + + fs.copyFile(from, to, (error) => { + if (error) { + // eslint-disable-next-line no-console + console.error(`Error copying file: ${error.message}`); + } + }); +} + +const params = process.argv.slice(2); + +if (params.length > 0 && params.length < 2) { + // eslint-disable-next-line no-console + console.error('Usage: node app.js '); +} else { + const src = params[0]; + const dest = params[1]; + + copyFile(src, dest); +} From b76b3c738672ec743c5a570c658de844c528cc2f Mon Sep 17 00:00:00 2001 From: Les Dyman Date: Mon, 9 Feb 2026 12:19:20 +0100 Subject: [PATCH 2/2] fix issue with no arguments --- src/app.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/app.js b/src/app.js index 93b6a76..7ab3e04 100644 --- a/src/app.js +++ b/src/app.js @@ -26,7 +26,7 @@ function copyFile(from, to) { const params = process.argv.slice(2); -if (params.length > 0 && params.length < 2) { +if (params.length !== 2) { // eslint-disable-next-line no-console console.error('Usage: node app.js '); } else {