Skip to content

Use Node 10 and npm 6 for development #2934

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

Merged
merged 13 commits into from
Aug 23, 2018
Merged

Use Node 10 and npm 6 for development #2934

merged 13 commits into from
Aug 23, 2018

Conversation

etpinard
Copy link
Contributor

... and Chrome 68 on CI.

Node 10 will become LTS in October, but why wait?

Switching to Node 10 makes us the latest major npm version with improved package-lock and faster installs and the latest v8 engine, speeding up our builds and CI runs (e.g. the build job on CI runs 30s faster, syntax is 30s faster too).

The switch to Chrome 68 appears to make two tests consistently fail:

image

image

The second one is probably related to #2747 (we might need to put a @noCI tag on that one).

@etpinard
Copy link
Contributor Author

cc @alexcjohnson

@alexcjohnson
Copy link
Collaborator

Huh... the first test failure is just a slightly different rounding result - should be able to just increase the tolerance. But the second, we still have the explicit timezone-setting step:
screen shot 2018-08-21 at 2 09 38 pm
But this is apparently not being honored by Chrome? There must be some other way to get it to use a timezone...

I see two timezone switches in the chrome flags:
--disable-per-user-timezone
--disable-system-timezone-automatic-detection
Hard to tell if one of those would get us back to the previous behavior of "just look at /etc/localtime" but we could try.

Actually, looking at where that page got its info from, maybe this is the new one that would help us:
--disable-fine-grained-time-zone-detection

@alexcjohnson
Copy link
Collaborator

Whew, fixing the timezone issue was harder than it should have been, thanks for persevering there! This looks great. 💃
Oddly though, it seems like a wash in terms of total test time - build is faster but test-jasmine and test-jasmine2 are both slower... oh well, still better to be up to date.

@etpinard
Copy link
Contributor Author

but test-jasmine and test-jasmine2 are both slower.

Interesting. I guess Chrome 68 is slower than Chrome 64?

@etpinard etpinard merged commit bff736d into master Aug 23, 2018
@etpinard etpinard deleted the node10 branch August 23, 2018 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants