Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: #19 #21

Merged
merged 1 commit into from
Oct 23, 2019
Merged

fix: #19 #21

merged 1 commit into from
Oct 23, 2019

Conversation

lxfriday
Copy link
Collaborator

@lxfriday lxfriday commented Oct 21, 2019

ref #19

@codecov
Copy link

codecov bot commented Oct 21, 2019

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@         Coverage Diff          @@
##           master   #21   +/-   ##
====================================
  Coverage      96%   96%           
====================================
  Files           1     1           
  Lines          50    50           
  Branches       12    12           
====================================
  Hits           48    48           
  Misses          2     2
Impacted Files Coverage Δ
src/index.ts 96% <100%> (ø) ⬆️

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 8d2c624...cb5560d. Read the comment docs.

Copy link
Member

@camsong camsong left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks

@camsong camsong merged commit 6d29b95 into nefe:master Oct 23, 2019
@camsong
Copy link
Member

camsong commented Oct 23, 2019

@lxfriday Thanks for your contributions 👏 I would like to invite you to be a collaborator.
Please add my wechat tel: 13162473297

@lxfriday
Copy link
Collaborator Author

@camsong Thank you, it's an honor to me.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants