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

tools: include IANA TLD URL in new gtld updates. #1817

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

cpu
Copy link
Contributor

@cpu cpu commented Aug 3, 2023

This branch updates the tools/newgtlds.go tooling to include a comment line with the gTLD's IANA registry URL for each gTLD.

E.g. for the tld aaa, includes a comment line: // https://www.iana.org/domains/root/db/aaa.html.

Follow-up to #1812

This commit updates the `tools/newgtlds.go` tooling to include a comment
line with the gTLD's IANA registry URL for each gTLD.

E.g. for the tld `aaa`, includes a comment line:
`// https://www.iana.org/domains/root/db/aaa.html`
@dnsguru dnsguru self-assigned this Aug 4, 2023
@dnsguru dnsguru requested a review from weppos August 4, 2023 14:16
@dnsguru dnsguru added ✅ autopull Automation pull from Authoritative ICANN json source 🚩ICANN (IANA/ICP-3) Section PR changes in the ICANN/IANA section typically reserved for TLDs. non .dat Change or Coding Review Alteration to code/automation or publicsuffix.org site labels Aug 4, 2023
Copy link
Member

@dnsguru dnsguru left a comment

Choose a reason for hiding this comment

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

LGTM

@dnsguru dnsguru merged commit c0a861f into publicsuffix:master Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ autopull Automation pull from Authoritative ICANN json source 🚩ICANN (IANA/ICP-3) Section PR changes in the ICANN/IANA section typically reserved for TLDs. non .dat Change or Coding Review Alteration to code/automation or publicsuffix.org site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants