Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on May 26, 2023. It is now read-only.

feat/support-node-14 #276

Merged
merged 14 commits into from
Feb 5, 2021
Merged

feat/support-node-14 #276

merged 14 commits into from
Feb 5, 2021

Conversation

teye
Copy link
Contributor

@teye teye commented Jan 29, 2021

This pr updates Node version support from 12 to 14 (latest LTS).

This pr also includes minor updates to some of the packages in order to resolve yarn bundle issues on Windows on Node 14:

  • @types/node 12.7.12 -> 14.14.22
  • protobufjs 6.8.8 -> 6.10.2

This pr also removes the need for web3-eth to be installed. web3-eth was used previously to generate a keystores.json file to used for testcases for zilliqa-js-account. The new test accounts are now generated using zilliqa-js/crypto to a new keystores_redux.json file.

Coincidentally, the removal of web3-eth also removes the requirement for a js-sha3 module which in turns removes the need for Visual Studio Build Tools to be installed on Windows when performing a yarn install.

The branch is tested with the following node LTS versions / functions and on both Windows 10 10.0.19041 Build 19041 and Ubuntu 18.04 :

  • Node 14.15.4 / 12.20.1 / 10.23.1
  • yarn install
  • yarn bootstrap
  • yarn test
  • yarn build:web
  • yarn bundle
  • examples/webpack/html
  • examples/webpack/react

@teye teye requested review from bb111189, micovi and renlulu January 29, 2021 11:18
@teye teye self-assigned this Jan 29, 2021
@teye teye requested a review from AmritKumar as a code owner January 29, 2021 11:18
@bb111189 bb111189 requested a review from hicaru February 1, 2021 03:21
@hicaru
Copy link

hicaru commented Feb 1, 2021

For migrate nodejs 14 need change package.json too.

@bb111189
Copy link
Contributor

bb111189 commented Feb 1, 2021

help me rename
"Zilliqa Usability Team [email protected] (https://github.com/zilliqa/)" to Zilliqa Usability Team [email protected] (https://github.com/zilliqa/)

@teye
Copy link
Contributor Author

teye commented Feb 1, 2021

@codecov-io
Copy link

codecov-io commented Feb 1, 2021

Codecov Report

Merging #276 (9c51526) into dev (db97fd7) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #276   +/-   ##
=======================================
  Coverage   77.93%   77.93%           
=======================================
  Files          45       45           
  Lines        1677     1677           
  Branches      271      271           
=======================================
  Hits         1307     1307           
  Misses        366      366           
  Partials        4        4           
Impacted Files Coverage Δ
packages/zilliqa/src/index.ts 0.00% <0.00%> (ø)

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 db97fd7...9c51526. Read the comment docs.

@bb111189
Copy link
Contributor

bb111189 commented Feb 3, 2021

Due to webpack changes, @teye will need you to fix conflict and regenerate the min.js

@teye teye requested a review from snowsledge as a code owner February 4, 2021 03:09
@bb111189
Copy link
Contributor

bb111189 commented Feb 5, 2021

@teye can also update https://github.com/Zilliqa/Zilliqa-JavaScript-Library/blob/dev/.travis.yml to have node14 so that the travis and build and test it?

@bb111189 bb111189 merged commit 672b721 into dev Feb 5, 2021
@bb111189 bb111189 deleted the feat/upgrade-node-14 branch August 18, 2021 05:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants