Skip to content
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

Move docker images to use node v10 #1183

Merged
merged 2 commits into from
Oct 2, 2019
Merged

Conversation

ashishb
Copy link
Contributor

@ashishb ashishb commented Oct 2, 2019

Description

Move docker images to use node v10

Tested

Google Cloud build is building and testing the changes.
Locally tested one file $ docker build -f dockerfiles/celotool/Dockerfile .

@codecov
Copy link

codecov bot commented Oct 2, 2019

Codecov Report

Merging #1183 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1183      +/-   ##
==========================================
- Coverage   66.16%   66.06%   -0.11%     
==========================================
  Files         261      261              
  Lines        7599     7564      -35     
  Branches      440      504      +64     
==========================================
- Hits         5028     4997      -31     
+ Misses       2476     2469       -7     
- Partials       95       98       +3
Flag Coverage Δ
#mobile 66.06% <ø> (-0.11%) ⬇️
Impacted Files Coverage Δ
packages/mobile/src/qrcode/utils.ts 72.5% <0%> (-1.86%) ⬇️
packages/mobile/src/recipients/RecipientPicker.tsx 71.62% <0%> (-1.46%) ⬇️
packages/mobile/src/send/SendAmount.tsx 77.14% <0%> (-0.64%) ⬇️
packages/mobile/src/utils/formatting.ts 89.74% <0%> (ø) ⬆️
packages/mobile/src/qrcode/QRScanner.tsx 0% <0%> (ø) ⬆️
packages/mobile/src/identity/reducer.ts 41.66% <0%> (ø) ⬆️
packages/mobile/src/qrcode/QRGen.tsx
packages/mobile/src/send/LabeledTextInput.tsx 78.94% <0%> (ø)

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 b47728e...fe8bba7. Read the comment docs.

Copy link
Contributor

@cmcewen cmcewen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏 thanks! sorry missed this

@ashishb
Copy link
Contributor Author

ashishb commented Oct 2, 2019

🙏 thanks! sorry missed this

No worries, at some point, we will have to make Google cloud build a mandatory step as well.

@ashishb ashishb added the automerge Have PR merge automatically when checks pass label Oct 2, 2019
@ashishb ashishb merged commit 6bc3f92 into master Oct 2, 2019
@ashishb ashishb deleted the ashishb/docker_images_node_10 branch October 2, 2019 23:46
aaronmgdr added a commit that referenced this pull request Oct 4, 2019
* master:
  [Protocol] Fix network id for alfajores in truffle configs (#1211)
  When resetting and upgrading a VM testnet, new tx-nodes are included in the new instance group (#771)
  Upload static VM testnet nodes, add stackdriver logging (#750)
  Revert "Make packages depend on git vesrion (not npm)" (#1201)
  Make packages depend on git vesrion (not npm) (#1192)
  [contractkit] Document methods (#1195)
  [ck] consistent send tx object in kit (#1191)
  Move docker images to use node v10 (#1183)
  [ContractKit]Fill more fields before web3 signing (#1133)
  [codecov]Fix codecov errors (#1147)
  [Wallet] Add support for address pasting in send input field (#1180)
  Fix verification pool validation (#1176)
  Improve QR Code scan ability (#1036)
  Add CLI commands around identity metadata (#1167)
  [wallet]Run geth in an infura-like mode (#1108)

# Conflicts:
#	yarn.lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants