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

Release 8.2.0 #1222

Merged
merged 50 commits into from
Jun 17, 2021
Merged

Release 8.2.0 #1222

merged 50 commits into from
Jun 17, 2021

Conversation

mradkov
Copy link
Contributor

@mradkov mradkov commented Jun 17, 2021

8.2.0 (2021-06-17)

⚠ BREAKING CHANGES

  • crypto: remove outdated generateSaveWallet function
  • crypto: remove unused prepareTx, encodeTx, decodeTx functions
  • crypto: remove unused hexStringToByte function
  • crypto: rename messageToBinary to messageToHash adding hashing
  • crypto: drop extra "personal" from message functions
  • crypto: remove unused formatAddress function
  • crypto: remove unused addressToHex function
  • node-pool: inline helpers, export by default
  • string: use isAddressValid instead of isAeAddress
  • string: remove unused snakeOrKebabToPascal function
  • drop outdated protocols and transactions schemas
  • drop compatibility with node@5

Bug Fixes

  • export aepp-wallet-communication (d5d741c)
  • ponyfill Buffer in browser (953bf08)
  • ponyfill process in browser (798ab63)
  • rlp: import as it is (736b0f5)
  • typo name of broadcast failed error generator (ae7e823)
  • examlple-aepp: open only when ready to accept connection (4872eb9)

Refactoring

  • crypto: drop extra "personal" from message functions (34288cb)
  • crypto: remove outdated generateSaveWallet function (37298be)
  • crypto: remove unused addressToHex function (93f9def)
  • crypto: remove unused formatAddress function (a5d4b62)
  • crypto: remove unused hexStringToByte function (ed39a76)
  • crypto: remove unused prepareTx, encodeTx, decodeTx functions (64d15eb)
  • crypto: rename messageToBinary to messageToHash adding hashing (df37004)
  • node-pool: inline helpers, export by default (ed1cfb5)
  • string: remove unused snakeOrKebabToPascal function (79bdc04)
  • string: use isAddressValid instead of isAeAddress (ac7d827)
  • drop compatibility with node@5 (f5e2fdb)
  • drop outdated protocols and transactions schemas (f18d305)

davidyuk and others added 30 commits May 31, 2021 13:03
Bumps [ws](https://github.com/websockets/ws) from 7.4.5 to 7.4.6.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.4.5...7.4.6)

Signed-off-by: dependabot[bot] <support@github.com>
reverts 03c2dcb
node_modules/rlp/dist/index.js doesn't contain `const` anymore, but this processing was breaking import of commonjs module
Fixes error: ReferenceError: process is not defined
`process` referenced in pbkdf2 package
Fixes error: ReferenceError: Buffer is not defined
Fix lris updating domain name maximum height issue
refactor: validate nodes using Joi
refactor: drop compatibility with node@5
@mradkov mradkov requested a review from davidyuk June 17, 2021 07:16
@codecov
Copy link

codecov bot commented Jun 17, 2021

Codecov Report

Merging #1222 (ebcfeed) into master (1da63d7) will decrease coverage by 33.38%.
The diff coverage is 89.01%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #1222       +/-   ##
===========================================
- Coverage   93.59%   60.21%   -33.39%     
===========================================
  Files          54       56        +2     
  Lines        2609     2609               
  Branches       24       36       +12     
===========================================
- Hits         2442     1571      -871     
- Misses        155     1032      +877     
+ Partials       12        6        -6     
Impacted Files Coverage Δ
src/chain/node.js 70.17% <ø> (-24.57%) ⬇️
.../utils/aepp-wallet-communication/rpc/wallet-rpc.js 99.14% <ø> (ø)
src/utils/aepp-wallet-communication/schema.js 90.00% <0.00%> (ø)
src/utils/swagger.js 100.00% <ø> (ø)
src/contract/aci/transformation.js 6.39% <50.00%> (-88.96%) ⬇️
src/utils/crypto.js 57.64% <71.42%> (-27.81%) ⬇️
src/utils/bytes.ts 66.66% <75.00%> (ø)
src/utils/string.ts 70.00% <77.77%> (ø)
src/account/memory.js 95.65% <92.85%> (-4.35%) ⬇️
src/account/base.js 100.00% <100.00%> (ø)
... and 36 more

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 1da63d7...ebcfeed. Read the comment docs.

@davidyuk
Copy link
Member

@mradkov did you used the new npm run release script? It will work fine if create a local tag v8.1.0 at the place of 8.1.0. In the future, I'm proposing to switch to v-prefixed tags. Also, would be nice to merge other PRs before release.

@mradkov mradkov changed the title Rrelease 8.2.0 Release 8.2.0 Jun 17, 2021
@mradkov mradkov merged commit 849c875 into master Jun 17, 2021
@davidyuk davidyuk deleted the release/v8.2.0 branch June 17, 2021 12:52
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.

None yet

3 participants