-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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: Update to Electron 6.x #75802
Changes from 97 commits
438dbae
6f67412
eadf90e
24baed8
3e2137d
8b60801
e04d7ee
c236b98
b04ab8f
29dd76e
0e3fd52
0a42812
ee06c48
77b0afd
fa057c5
b37922a
5d58933
af4ccd3
7a1656c
4befda6
e6c902d
d366921
136e07d
f30ce7b
d4d924a
43c0e06
f269e25
f278d18
5d5d07e
8f2df7c
08e186f
4f87ca7
1f2f377
e9e87d1
9b5a6f6
9d42864
3f62931
a58b414
7b69832
a2df3c0
09a699e
5eb9925
ed3fee0
39e30e7
929c9ac
bca0338
c3f39af
c8f6695
627c367
a96d23e
86b3df8
b6afd0a
a8267b3
9385344
cb3deee
62783f0
da83170
5f3cbda
9702b9d
5026228
bca4cef
2eb49ce
180b77b
dd03761
8c2bb50
64be4dd
2424dae
0db4054
068cd21
83beb0e
7c4d03a
7173885
651e723
b35cdfc
a644054
f98d6c3
3e54481
0dd605c
a91e316
3949a86
54fa069
e541730
d83aabf
4124349
9695067
ceca6a7
5c5897d
2cdc661
52480a4
38963f9
7240526
b45c49e
b3d19e6
9711749
e058e95
f0d8470
6449c75
6fcc13d
69908c7
63793fd
da3fc09
84536f9
d0f648d
454b637
039ef75
e710d58
5d2aae8
9a55e7e
d1846d4
dd4f0b0
509bf8d
eeee33a
fdb0378
b30a73e
d6b1e32
905cf4e
c257f44
3f4b3e0
5d0ab28
9b73cba
65a0011
9a09f02
dc67f3c
b87aff9
9652a73
a782e63
56e5dbc
4ff085a
bccb589
918d9c7
9a46d44
60cd483
36250ad
5a394c5
7b55e3d
61bb49d
8f7dd73
cd82e68
b9c9b22
305ea70
d5923be
799e528
198ae70
b134e13
4490d98
8460cba
a58bfc7
f797c6e
81e3421
c82ac1c
5cafa4d
9eb8bb4
2912aa8
494add2
e0b27fb
e0e9dba
41ad7be
2cb45e1
8151a72
64ca757
babbc01
6ef7e17
2752944
292c83d
0cc12da
d8f6c88
71c0997
6a6eaa7
6ab2fe3
e2a5860
a742b44
d582bf5
fb8510f
32bb388
20907bd
3f74a02
068bcc0
29c1dca
7b6c570
2f43556
b455429
b001fb8
9ec6e85
2565eb6
f5ea48a
9cfce78
768ef49
825ac8c
2d2f577
cbbc0f6
a98c9ab
8ac4dde
58d86a0
9fe7c01
d30b3e9
2b4d2da
f66a27f
84e8011
ecfe646
0b23385
be78317
a456dd3
9009193
c87c535
58321b0
2728185
2379919
72c900d
4aceb31
2f25cf1
cfcadf2
6d961d0
3e0bf57
e81cd9a
dff5304
7195ff0
eb8e59d
af1be63
2d0f273
7f384d7
3af546e
1cf07fa
83a9288
822fe2f
539b331
a990865
4c6f203
ec61e4d
15ef98a
8af6f8c
cc4ab0e
de6f255
5a01469
eae73dc
56fba48
6011ec1
16f16a9
d1a13ae
12118a5
d92cfaf
2555e23
a715df3
254f683
17fbc66
7d97d83
8b9dc82
fe0becd
1f06282
e4158b7
c937d89
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
disturl "https://atom.io/download/electron" | ||
target "4.2.7" | ||
target "6.0.0" | ||
runtime "electron" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
2019-07-11T05:47:05.444Z | ||
2019-07-13T05:47:05.444Z | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,6 +6,9 @@ resources: | |
- container: snapcraft | ||
image: snapcore/snapcraft:stable | ||
|
||
variables: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @deepak1556 this should not be part of the squashed commit as it was only done for exploration build |
||
VSCODE_QUALITY: exploration | ||
|
||
jobs: | ||
- job: Compile | ||
pool: | ||
|
@@ -128,8 +131,8 @@ jobs: | |
- template: sync-mooncake.yml | ||
|
||
schedules: | ||
- cron: "0 5 * * Mon-Fri" | ||
displayName: Mon-Fri at 7:00 | ||
- cron: "0 6 * * Mon-Fri" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @deepak1556 this scheduling rule change should not be part of the squashed commit as it was only done for exploration build |
||
displayName: Mon-Fri at 8:00 | ||
branches: | ||
include: | ||
- master | ||
- electron-6.0.x |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,11 @@ | ||
steps: | ||
- script: | | ||
set -e | ||
git config user.email "[email protected]" | ||
git config user.name "VSCode" | ||
git merge origin/master | ||
displayName: Merge master | ||
|
||
- script: | | ||
mkdir -p .build | ||
echo -n $BUILD_SOURCEVERSION > .build/commit | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,7 +17,7 @@ const paths = require('./paths'); | |
// @ts-ignore | ||
const product = require('../product.json'); | ||
// @ts-ignore | ||
const app = require('electron').app; | ||
const { app, protocol } = require('electron'); | ||
|
||
// Enable portable support | ||
const portable = bootstrap.configurePortable(); | ||
|
@@ -33,6 +33,11 @@ app.setPath('userData', userDataPath); | |
// Update cwd based on environment and platform | ||
setCurrentWorkingDirectory(); | ||
|
||
// Register custom schemes with privileges | ||
protocol.registerSchemesAsPrivileged([ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @deepak1556 I suggest to follow up with @mjbvz , it seems to me we are using different options on master: require('electron').webFrame.registerURLSchemeAsPrivileged('vscode-resource', {
secure: true,
bypassCSP: false,
allowServiceWorkers: false,
supportFetchAPI: true,
corsEnabled: true
}); Also, since this happens on startup we got to be 100% sure this does not slow down our startup. Is it really necessary to do so early? |
||
{ scheme: 'vscode-resource', privileges: { secure: true, supportFetchAPI: true, corsEnabled: true } } | ||
]); | ||
|
||
// Global app listeners | ||
registerListeners(); | ||
|
||
|
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.
@deepak1556 this should not be part of the squashed commit as it was only done for exploration build