Skip to content

Comments

Bump electron to 6#48

Closed
aminya wants to merge 63 commits intomasterfrom
bump_electron_6
Closed

Bump electron to 6#48
aminya wants to merge 63 commits intomasterfrom
bump_electron_6

Conversation

@aminya
Copy link
Member

@aminya aminya commented Jul 18, 2020

We will not merge this. This is just an experiment, so do not review this.

Differences with upstream:
- I bumped vscode-ripgrep from 1.2.5 to 1.8: atom#21056

  • Etch bumped to 0.14
  • electron-link bumped
  • bumped settings view
  • bumped find and replace
  • package symbolic link PR merged
  • use apm node-12
  • bumped scrollbar style
  • We use clang 9 on Linux
  • Our CI is faster and up to date.

@aminya aminya mentioned this pull request Jul 18, 2020
@aminya
Copy link
Member Author

aminya commented Jul 18, 2020

Find and replace tests fail on this one. But I want to test them manually. Maybe the tests are out of date as well.

@aminya
Copy link
Member Author

aminya commented Jul 18, 2020

I downloaded the windows artifact from here: https://dev.azure.com/atomcommunity/atomcommunity/_build/results?buildId=317&view=artifacts&type=publishedArtifacts

Find and replace works for me in one file. It does not work with your instructions atom#21079 (comment)

However, in the settings view for one package, I cannot scroll using trackpad gestures.

parallelize core tests

allow requesting parallel tests for all OS

run tests using async.parallel

don't use temp.track()

cleanup process fails when the file is locked (in parallel tests)

warn before error reporting

Helps visually

test: use Azure format for printing

Run windows renderer tests in parallel

Run windows core main tests in the build step

always upload atom windows.zip for x64

bootstrap in case cache misses

macos: run core main tests in the build phase

print the used testCommand for failed tests

update message about finding a single application to run the tests
@aminya aminya force-pushed the bump_electron_6 branch from ddc5a4b to a1dcfc9 Compare July 18, 2020 06:08
@aminya aminya force-pushed the bump_electron_6 branch from 853e624 to bda4510 Compare July 18, 2020 06:42
@aminya aminya force-pushed the bump_electron_6 branch from b94de00 to 65bdd3f Compare July 18, 2020 07:02
@aminya aminya marked this pull request as draft July 19, 2020 18:24
@aminya
Copy link
Member Author

aminya commented Jul 22, 2020

I cannot search for packages for downloading in the settings view. I wonder if this is caused by the recent settings view change.

@DeeDeeG Can you reproduce this with the upstream branch instead?

image

@aminya
Copy link
Member Author

aminya commented Jul 22, 2020

I am going to close this as the upstream pull request is updated.

@aminya aminya closed this Jul 22, 2020
@DeeDeeG
Copy link
Member

DeeDeeG commented Jul 31, 2020

I am willing to merge Electron 6 here (atom-ide-community/atom) and deal with anything that comes up if it will help convince upstream to merge it.

Yet, is complicates our other PRs. But somebody's going to have to do it eventually. I think if we want to show upstream we're serious, this could be something to do here first.

Any thoughts? Agree/disagree? (I wonder why upstream isn't merging it anyhow... Hmm. It seems ready to me.)

Edit: If you want, we can just have a branch that's the "Electron 6 Take Two" PR floated (periodically rebased or master merged into it) on top of our master. I wouldn't mind seeing how that goes.

Bonus: lets us do an experimental Electron 7 PR and start finding and clearing up the barriers to Electron 7 for ourselves and upstream when we get the time.

@aminya
Copy link
Member Author

aminya commented Jul 31, 2020

I was thinking about that. Let me do it myself. I will delete some of these useless closed branches and will merge it!

I love the Electron 7 idea! My next big thing is #67. You can work on electron 7 in parallel.

Edit: Add the things you are working to Projects. I want us to keep track of the things there so things do not get lost even if we close pull requests.

@aminya aminya deleted the bump_electron_6 branch July 31, 2020 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants