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

Added Posh-ACME to client list #78

Merged
merged 2 commits into from
May 2, 2018
Merged

Added Posh-ACME to client list #78

merged 2 commits into from
May 2, 2018

Conversation

rmbolger
Copy link
Contributor

I just merged the first version of the acme-dns plugin for Posh-ACME via rmbolger/Posh-ACME#4. So it's in the master branch, but I'm hoping to get a few more plugins written before making a new official release. I understand if you want to wait to accept this PR until that happens.

@coveralls
Copy link

coveralls commented Apr 30, 2018

Coverage Status

Coverage remained the same at 92.244% when pulling dd5dd25 on rmbolger:master into a17792d on joohoi:master.

@joohoi
Copy link
Owner

joohoi commented Apr 30, 2018

If you feel that it's ready, we can add it. I would prefer the client list to be in alphabetical order however.

@rmbolger
Copy link
Contributor Author

It's definitely working. But it's probably best to wait for the release so that people getting the client through the normal PowerShell Gallery channel will have the plugin.

@joohoi
Copy link
Owner

joohoi commented Apr 30, 2018

Ok, sounds good. LMK when it's ready.

@rmbolger
Copy link
Contributor Author

rmbolger commented May 2, 2018

Posh-ACME 1.1 is now released to the PowerShell Gallery and contains acme-dns support. All good to merge this.

@joohoi joohoi merged commit fde566f into joohoi:master May 2, 2018
@joohoi
Copy link
Owner

joohoi commented May 2, 2018

Nice! Merged

jacobmyers-codeninja pushed a commit to jacobmyers-codeninja/acme-dns that referenced this pull request Sep 30, 2020
* Added Posh-ACME to client list

* alphabetical order as requested
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants