Skip to content
This repository was archived by the owner on Mar 11, 2026. It is now read-only.

chore(build): use the latest npm on windows for tests#304

Merged
jkwlui merged 2 commits intogoogleapis:masterfrom
JustinBeckwith:lolwindows
Nov 7, 2018
Merged

chore(build): use the latest npm on windows for tests#304
jkwlui merged 2 commits intogoogleapis:masterfrom
JustinBeckwith:lolwindows

Conversation

@JustinBeckwith
Copy link
Copy Markdown
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 2, 2018
@JustinBeckwith
Copy link
Copy Markdown
Contributor Author

@kinwa91 for whatever reason, this repo is seeing some weird failures with npm install on Windows. It looks like upgrading to the latest version of npm seems to fix things 🤷‍♂️

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 7, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@02503ab). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #304   +/-   ##
=========================================
  Coverage          ?   92.19%           
=========================================
  Files             ?       14           
  Lines             ?      641           
  Branches          ?       57           
=========================================
  Hits              ?      591           
  Misses            ?       36           
  Partials          ?       14

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02503ab...6c29ae3. Read the comment docs.

@jkwlui
Copy link
Copy Markdown
Contributor

jkwlui commented Nov 7, 2018

We should probably apply the change across all repos if we think the current version is bad - I can @google/repo it.

@jkwlui jkwlui merged commit 9bbe986 into googleapis:master Nov 7, 2018
@JustinBeckwith
Copy link
Copy Markdown
Contributor Author

I mean I don't know if what I'm doing here makes a difference 😆 but it seemed to work

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants