Skip to content

Conversation

@gxcsoccer
Copy link
Member

No description provided.

@codecov-io
Copy link

codecov-io commented May 15, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@3641e89). Click here to learn what that means.
The diff coverage is 94.69%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #35   +/-   ##
=========================================
  Coverage          ?   94.26%           
=========================================
  Files             ?        2           
  Lines             ?      366           
  Branches          ?        0           
=========================================
  Hits              ?      345           
  Misses            ?       21           
  Partials          ?        0
Impacted Files Coverage Δ
lib/number.js 100% <100%> (ø)
lib/byte.js 94.24% <94.64%> (ø)

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 3641e89...f534dff. Read the comment docs.

@gxcsoccer gxcsoccer force-pushed the not-support-old-node branch from 48736c1 to db6e0f7 Compare May 15, 2018 08:51
@fengmk2
Copy link
Member

fengmk2 commented May 15, 2018

@gxcsoccer please change your commit author

image

const size = handles.size;

ByteBuffer.prototype[putMethod] = function (index, value) {
ByteBuffer.prototype[putMethod] = function(index, value) {
Copy link
Member

Choose a reason for hiding this comment

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

use class style instead of prototype?

Copy link
Member

@fengmk2 fengmk2 left a comment

Choose a reason for hiding this comment

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

ignore me

@gxcsoccer gxcsoccer force-pushed the not-support-old-node branch from db6e0f7 to b30c752 Compare May 16, 2018 03:01
@gxcsoccer gxcsoccer force-pushed the not-support-old-node branch from b30c752 to f534dff Compare May 16, 2018 03:05
@gxcsoccer gxcsoccer merged commit dc4f5a6 into master May 16, 2018
@gxcsoccer gxcsoccer deleted the not-support-old-node branch May 16, 2018 03:15
@gxcsoccer
Copy link
Member Author

2.0.0

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants