-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Fix RFC2316 Windows Documentation #1516
Fix RFC2316 Windows Documentation #1516
Conversation
Welcome @scottd018! |
@scottd018 thanks, do you mind changing the commit message? |
@njuettner Yep, I'm happy to fix it. One quick, ignorant question...what format do the commit messages need to be in? Apologies as this is my first time committing to this project. I looked through the contributing guidelines, and either I'm overlooking it, or I don't happen to see what the expected format is. Let me know and I'll get it fixed for you right away. Thanks! |
@njuettner Give the new commit message a look. I am still having trouble finding what is proper. If it's wrong, I'll give it another try. Thanks. |
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: njuettner, scottd018 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@scottd018 thanks again for your PR and sorry for the late response. I saw you already figured it out 👍 |
@njuettner no problem! thanks for the merge and the patience with new contributors. :) |
…-documentation-rfc2136 Fix RFC2316 Windows Documentation
Added documentation to show the use of the
--rfc2136-insecure
flag when integrating with Windows DNS when using insecure zone updates.