Skip to content
This repository has been archived by the owner on Dec 19, 2023. It is now read-only.

Properly group records from a zonefile. #26

Merged
merged 3 commits into from
Mar 15, 2018

Conversation

stephenplusplus
Copy link
Contributor

Fixes #8

cc @timwsuqld -- would you mind giving this a try?

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 14, 2018
@ghost ghost assigned stephenplusplus Mar 14, 2018
@codecov
Copy link

codecov bot commented Mar 14, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           4      4           
  Lines         250    264   +14     
=====================================
+ Hits          250    264   +14
Impacted Files Coverage Δ
src/zone.js 100% <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 2807aee...d742e29. Read the comment docs.

{DNS_DOMAIN} A 128.9.0.32
A 10.1.0.52
{DNS_DOMAIN} A 10.2.0.27
A 128.9.0.33

This comment was marked as spam.

This comment was marked as spam.

@stephenplusplus stephenplusplus merged commit 67e2df6 into googleapis:master Mar 15, 2018
@ghost ghost removed the cla: yes This human has signed the Contributor License Agreement. label Mar 15, 2018
@stephenplusplus stephenplusplus mentioned this pull request Mar 15, 2018
@timwsuqld
Copy link

@stephenplusplus Sorry. I worked around this issue by using another tool, so don't have anything handy to test with. Thanks for the fix though. When I have more time I'll revist where I had wanted to use this, as my current work around is a little hacky

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.

None yet

4 participants