-
-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: upgrade electron to v22 #720
Merged
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
9076ec2
chore: upgraded electron and node
ErikSin aa774f2
bring back map server using latest
achou11 3184ed1
replace electron.remote with @electron/remote
achou11 6ec4819
fix usage of electron-is-dev module
achou11 ba81279
add patch for ecstatic
achou11 5ce26c8
remove duplicate code in app.js
achou11 5248e38
fix import of electron remote in renderer
achou11 3d06334
remove electron remote from externals
achou11 b1c0c25
fix issue with creating map server db before directory exists
achou11 5cc4c5b
fix mapServer scoping error when quitting app
achou11 364241b
upgrade react-intl to remove usage of broken peer dep
achou11 d92a821
remove unused livereload functionality
achou11 536c002
remove commented-out webpack plugin
achou11 c548a1a
update comment in new api file
achou11 9ea06e0
adjustments for window.mode in background windows
achou11 d78ae70
upgrade electron to 22.2.0
achou11 4278c40
ci changes
achou11 1d62322
Merge branch 'master' into es/chore-electron-upgrade
achou11 39fabc8
maybe fix electron build issue with fsevents
achou11 d230364
maybe fix rimraf issue on windows ci
achou11 666ba8f
partially undo previous
achou11 9e9e37d
try node script for ci artifact removal
achou11 461bf68
upgrade map server to latest
achou11 3f98e7d
upgrade electron to latest 22
achou11 41f5a65
update babel config
achou11 c730f47
Merge branch 'master' into es/chore-electron-upgrade
achou11 709fa4d
update electron to latest v22
achou11 fa7bd87
upgrade electron to v22.3.2
achou11 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
v12.14.1 | ||
v16.17.1 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
#!/usr/bin/env node | ||
|
||
// This script removes unnecessary artifacts that we don't want to keep around after electron builder runs on CI | ||
|
||
const rimraf = require('rimraf') | ||
|
||
const variant = process.argv[2] | ||
|
||
if (!variant) { | ||
console.error(new Error('Must specify variant')) | ||
process.exit(1) | ||
} | ||
|
||
const files = `dist/${variant}/!(*.exe|*.deb|*.AppImage|*.dmg|*.yml|*.zip|github)` | ||
|
||
console.log(`Attempting to remove: ${files}`) | ||
|
||
rimraf(files, err => { | ||
if (err) { | ||
console.error(err) | ||
process.exit(1) | ||
} else { | ||
console.log('Success!') | ||
process.exit(0) | ||
} | ||
}) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replaced the statically defined constants with these functions because this file is run in various contexts that are sensitive to the environment e.g. main process, background renderer, primary window renderer