Assorted Desktop App repo fixes (#3065)

* Fix ESLint and turn on the check in CI

* npm audit fix

* Remove electron-rebuild, use electron-builder for robotjs

* Add mochawesome-report to eslintignore

* Remove linebreak-style

* Revert "Remove electron-rebuild, use electron-builder for robotjs"

This reverts commit be35eb7300df725e381538dd30446a9acb8ab1df.
This commit is contained in:
Devin Binnie 2024-06-14 11:46:35 -04:00 committed by GitHub
parent bf07674d68
commit be182ac513
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 21 additions and 16 deletions

View file

@ -1,3 +1,4 @@
node_modules
dist
api-types/lib
api-types/lib
mochawesome-report

View file

@ -29,6 +29,7 @@
"react/no-find-dom-node": 2,
"no-underscore-dangle": ["warn"],
"@mattermost/use-external-link": 0,
"linebreak-style": 0,
"import/order": [
2,
{

View file

@ -11,6 +11,9 @@ inputs:
runs:
using: "composite"
steps:
- name: ci/run-eslint
run: npm run lint:js-quiet
shell: ${{ inputs.shell }}
- name: ci/run-check-types
run: npm run check-types
shell: ${{ inputs.shell }}

20
package-lock.json generated
View file

@ -8011,12 +8011,12 @@
}
},
"node_modules/braces": {
"version": "3.0.2",
"resolved": "https://registry.npmjs.org/braces/-/braces-3.0.2.tgz",
"integrity": "sha512-b8um+L1RzM3WDSzvhm6gIz1yfTbBt6YTlcEKAvsmqCZZFw46z626lVj9j1yEPW33H5H+lBQpZMP1k8l+78Ha0A==",
"version": "3.0.3",
"resolved": "https://registry.npmjs.org/braces/-/braces-3.0.3.tgz",
"integrity": "sha512-yQbXgO/OSZVD2IsiLlro+7Hf6Q18EJrKSEsdoMzKePKXct3gvD8oLcOQdIzGupr5Fj+EDe8gO/lxc1BzfMpxvA==",
"dev": true,
"dependencies": {
"fill-range": "^7.0.1"
"fill-range": "^7.1.1"
},
"engines": {
"node": ">=8"
@ -9661,9 +9661,9 @@
"integrity": "sha512-I88TYZWc9XiYHRQ4/3c5rjjfgkjhLyW2luGIheGERbNQ6OY7yTybanSpDXZa8y7VUP9YmDcYa+eyq4ca7iLqWA=="
},
"node_modules/ejs": {
"version": "3.1.9",
"resolved": "https://registry.npmjs.org/ejs/-/ejs-3.1.9.tgz",
"integrity": "sha512-rC+QVNMJWv+MtPgkt0y+0rVEIdbtxVADApW9JXrUVlzHetgcyczP/E7DJmWJ4fJCZF2cPcBk0laWO9ZHMG3DmQ==",
"version": "3.1.10",
"resolved": "https://registry.npmjs.org/ejs/-/ejs-3.1.10.tgz",
"integrity": "sha512-UeJmFfOrAQS8OJWPZ4qtgHyWExa088/MtK5UEyoJGFH67cDEXkZSviOiKRCZ4Xij0zxI3JECgYs3oKx+AizQBA==",
"dev": true,
"dependencies": {
"jake": "^10.8.5"
@ -11600,9 +11600,9 @@
}
},
"node_modules/fill-range": {
"version": "7.0.1",
"resolved": "https://registry.npmjs.org/fill-range/-/fill-range-7.0.1.tgz",
"integrity": "sha512-qOo9F+dMUmC2Lcb4BbVvnKJxTPjCm+RRpe4gDuGrzkL7mEVl/djYSu2OdQ2Pa302N4oqkSg9ir6jaLWJ2USVpQ==",
"version": "7.1.1",
"resolved": "https://registry.npmjs.org/fill-range/-/fill-range-7.1.1.tgz",
"integrity": "sha512-YsGpe3WHLK8ZYi4tWDg2Jy3ebRz2rXowDxnld4bkQB00cc/1Zw9AWnC0i9ztDJitivtQvaI9KaLyKrc+hBW0yg==",
"dev": true,
"dependencies": {
"to-regex-range": "^5.0.1"

View file

@ -1,14 +1,14 @@
// Copyright (c) 2016-present Mattermost, Inc. All Rights Reserved.
// See LICENSE.txt for license information.
var fs = require('fs');
var path = require('path');
const fs = require('fs');
const path = require('path');
exports.default = async function beforePack(context) {
// The debian packager (fpm) complains when the directory to output the package to doesn't exist
// So we have to manually create it first
var dir = path.join(context.outDir, context.packager.appInfo.version)
if (!fs.existsSync(dir)){
const dir = path.join(context.outDir, context.packager.appInfo.version);
if (!fs.existsSync(dir)) {
fs.mkdirSync(dir, {recursive: true});
}
};

View file

@ -3,4 +3,4 @@
const fs = require('fs');
fs.copyFileSync('release/win-unpacked/resources/app-update.yml', 'release/msi-app-update.yml');
fs.unlinkSync('release/win-unpacked/resources/app-update.yml');
fs.unlinkSync('release/win-unpacked/resources/app-update.yml');