phlux/mattermostest
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[MM-37055] Remove commit hash from release builds generated by CircleCI (#1750)

Browse Source
This commit is contained in:
Devin Binnie
2021-09-22 06:17:57 -04:00
committed by GitHub
parent 92b297c469
commit e8523067b0
4 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/CriticalErrorHandler.ts
View File

@@ -19,7 +19,7 @@ function createErrorReport(err: Error) {
// eslint-disable-next-line no-undef // eslint-disable-next-line no-undef
// eslint-disable-next-line @typescript-eslint/ban-ts-comment // eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore // @ts-ignore
return `Application: ${app.name} ${app.getVersion()} [commit: ${__HASH_VERSION__}]\n` + return `Application: ${app.name} ${app.getVersion()}${__HASH_VERSION__ ? ` [commit: ${__HASH_VERSION__}]` : ''}\n` +
`Platform: ${os.type()} ${os.release()} ${os.arch()}\n` + `Platform: ${os.type()} ${os.release()} ${os.arch()}\n` +
`${err.stack}`; `${err.stack}`;
} }

2
src/main/menus/app.ts
View File

@@ -265,7 +265,7 @@ function createTemplate(config: Config) {
// eslint-disable-next-line no-undef // eslint-disable-next-line no-undef
// eslint-disable-next-line @typescript-eslint/ban-ts-comment // eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore // @ts-ignore
label: `Version ${app.getVersion()} commit: ${__HASH_VERSION__}`, label: `Version ${app.getVersion()}${__HASH_VERSION__ ? ` commit: ${__HASH_VERSION__}` : ''}`,
enabled: false, enabled: false,
}); });

2
src/renderer/index.tsx
View File

@@ -136,7 +136,7 @@ window.ipcRenderer.invoke('get-app-version').then(({name, version}) => {
// eslint-disable-next-line no-undef // eslint-disable-next-line no-undef
// eslint-disable-next-line @typescript-eslint/ban-ts-comment // eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore // @ts-ignore
console.log(`Starting ${name} v${version} commit: ${__HASH_VERSION__}`); console.log(`Starting ${name} v${version}${__HASH_VERSION__ ? ` commit: ${__HASH_VERSION__}` : ''}`);
}); });
ReactDOM.render( ReactDOM.render(

3
webpack.config.base.js
View File

@@ -14,9 +14,10 @@ const path = require('path');
const VERSION = childProcess.execSync('git rev-parse --short HEAD').toString(); const VERSION = childProcess.execSync('git rev-parse --short HEAD').toString();
const isProduction = process.env.NODE_ENV === 'production'; const isProduction = process.env.NODE_ENV === 'production';
const isRelease = process.env.CIRCLE_BRANCH && process.env.CIRCLE_BRANCH.startsWith('release-');
const codeDefinitions = { const codeDefinitions = {
__HASH_VERSION__: JSON.stringify(VERSION), __HASH_VERSION__: !isRelease && JSON.stringify(VERSION),
}; };
codeDefinitions['process.env.NODE_ENV'] = JSON.stringify(process.env.NODE_ENV); codeDefinitions['process.env.NODE_ENV'] = JSON.stringify(process.env.NODE_ENV);