Skip to content

Commit f93378a

Browse files
committed
fix: incorporate the review from @Totto16
1 parent dda2056 commit f93378a

File tree

5 files changed

+284
-444
lines changed

5 files changed

+284
-444
lines changed

flatpak-builder/dist/index.js.LICENSE.txt

Lines changed: 0 additions & 85 deletions
This file was deleted.

flatpak-builder/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -391,11 +391,11 @@ const run = async (config) => {
391391
return
392392
}
393393

394-
const artifact = new DefaultArtifactClient()
394+
const artifactClient = new DefaultArtifactClient()
395395
core.info('Uploading artifact...')
396396
// Append the arch to the bundle name to prevent conflicts in multi-arch jobs
397397
const bundleName = config.bundle.replace('.flatpak', '') + `-${config.arch}`
398-
return artifact.uploadArtifact(bundleName, [config.bundle], '.', {
398+
return artifactClient.uploadArtifact(bundleName, [config.bundle], '.', {
399399
continueOnError: false
400400
})
401401
})

flatpak-builder/package.json

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@
33
"main": "index.js",
44
"license": "MIT",
55
"scripts": {
6-
"test": "jest",
7-
"build": "webpack --progress --config webpack.prod.js"
6+
"test": "jest"
87
},
98
"jest": {
109
"verbose": true,
@@ -21,11 +20,11 @@
2120
"js-yaml": "^4.1.0"
2221
},
2322
"devDependencies": {
24-
"eslint": "^9.19.0",
23+
"eslint": "^8.49.0",
2524
"eslint-config-standard": "^17.1.0",
26-
"eslint-plugin-import": "^2.31.0",
27-
"eslint-plugin-jest": "^28.11.0",
28-
"eslint-plugin-n": "^17.15.1",
25+
"eslint-plugin-import": "^2.28.1",
26+
"eslint-plugin-jest": "^27.2.1",
27+
"eslint-plugin-n": "^16.0.2",
2928
"eslint-plugin-node": "^11.1.0",
3029
"eslint-plugin-promise": "^7.2.1",
3130
"jest": "^29.7.0",

flatpak-builder/webpack.prod.js

Lines changed: 0 additions & 12 deletions
This file was deleted.

0 commit comments

Comments
 (0)