-
Notifications
You must be signed in to change notification settings - Fork 234
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
added custom ca provider #283
base: master
Are you sure you want to change the base?
Conversation
edited
@@ -1,4 +1,5 @@ | |||
//+build !test | |||
//go:build !test |
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.
This syntax is for go1.17 and later.
Given that acme-dns still claims to compile with go1.13, the original syntax should be maintained.
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.
this was done automaticlly by the IDE/go tools, i think the compatiblity should be ok since line 2 has the old syntax. I think its good to have both syntax supported since the old one may gets removed one day.
Co-authored-by: Matthias Schneider <ms@wck.biz> Signed-off-by: İlteriş Yağıztegin Eroğlu <me@linuxgemini.space>
Co-authored-by: Matthias Schneider <ms@wck.biz> Co-authored-by: Johannes Doll <jodoll@users.noreply.github.com> Signed-off-by: İlteriş Yağıztegin Eroğlu <me@linuxgemini.space>