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

Simplify AliasForm wording #118

Open
wants to merge 4 commits into
base: master
from
Open

Simplify AliasForm wording #118

wants to merge 4 commits into from

Conversation

@bemasc
Copy link
Collaborator

bemasc commented Mar 9, 2020

The previous phrasing was redundant.

bemasc added 2 commits Mar 9, 2020
The previous phrasing was redundant.
Copy link

marka63 left a comment

"full-qualified" is redundant in a DNS record. All domain names in DNS records are fully qualified.

Additionally are you really wanting to tell master file parsers that they can't use a relative name in the master file.

@ 3600 IN SVBC foo

should be perfectly fine in a master file.

bemasc added 2 commits Mar 10, 2020
Remove redundant requirement on AliasForm values
@bemasc

This comment has been minimized.

Copy link
Collaborator Author

bemasc commented Mar 10, 2020

@ 3600 IN SVBC foo

should be perfectly fine in a master file.

Good point. I've updated the PR to allow that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.