Skip to content
This repository has been archived by the owner on Apr 10, 2024. It is now read-only.

Feat: Add getRecordType method to NsRecord [NS-1969] #24

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

simon-mycs
Copy link
Contributor

Missing method needed for NS-1969.

@codecov-io
Copy link

codecov-io commented Jan 17, 2019

Codecov Report

Merging #24 into master will increase coverage by 0.09%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
+ Coverage   75.53%   75.63%   +0.09%     
==========================================
  Files          34       34              
  Lines        1067     1067              
  Branches       68       69       +1     
==========================================
+ Hits          806      807       +1     
+ Misses        227      226       -1     
  Partials       34       34

lukaszMycs
lukaszMycs previously approved these changes Jan 17, 2019
Copy link
Contributor

@lukaszMycs lukaszMycs left a comment

Choose a reason for hiding this comment

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

looks good ;)

@lukaszMycs
Copy link
Contributor

It looks ok - I think you need to change version so that you can update it on npm...

Copy link
Contributor

@bbenzano bbenzano left a comment

Choose a reason for hiding this comment

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

Looks good to me

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