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

implement the NAPTR record #89

Merged
merged 10 commits into from Mar 27, 2023
Merged

implement the NAPTR record #89

merged 10 commits into from Mar 27, 2023

Conversation

xuhjbj
Copy link
Contributor

@xuhjbj xuhjbj commented Mar 20, 2023

Why

There is a requirement for support NAPTR record type.
So open a issue #87 to address.

What

Pull Request for this with @silverwind suggestion .

@silverwind
Copy link
Collaborator

silverwind commented Mar 20, 2023

Please add it to "Supported Record Types" in README.md, otherwise I think it looks good.

@xuhjbj
Copy link
Contributor Author

xuhjbj commented Mar 21, 2023

we had updated the README.md,please check.thanks

@xuhjbj
Copy link
Contributor Author

xuhjbj commented Mar 24, 2023

please review,and merge .Thanks.

index.js Outdated Show resolved Hide resolved
index.js Outdated Show resolved Hide resolved
index.js Outdated Show resolved Hide resolved
@silverwind
Copy link
Collaborator

I fixed lint, but tests are failing, please check.

@xuhjbj
Copy link
Contributor Author

xuhjbj commented Mar 27, 2023

I fixed the issue,please help review ,thanks.

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@silverwind silverwind merged commit aca1ff7 into mafintosh:master Mar 27, 2023
1 check passed
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.

None yet

2 participants