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.

Sign up for GitHub

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

Jump to bottom

Help text for enable-https could be clearer #43

Closed
thomascobb opened this issue Sep 13, 2016 · 3 comments
Closed

Help text for enable-https could be clearer #43

thomascobb opened this issue Sep 13, 2016 · 3 comments
Labels
fix-in-beta

Comments

@thomascobb
Copy link
Contributor

I missed the fact that you could run this script without arguments to get a Let's Encrypt certificate. I will submit a PR that tries to improve this...
@thomascobb thomascobb mentioned this issue Sep 13, 2016
Merged
@MariusBluem
Copy link

PR merged. Closing. THX! 😉

@kyrofa
Copy link
Member

kyrofa commented Sep 17, 2016 •
edited

This bug is not fixed until the fix has been merged into master-- it has not (need to test and merge the other PR before making the next release). I suggest we leave the bug open until it has been fixed from the user's point of view. I've made a fix-committed tag to track such a status.

@kyrofa kyrofa reopened this Sep 17, 2016
@kyrofa
Copy link
Member

kyrofa commented Sep 17, 2016

Note that the issue will be closed automatically due to the commit messages requested in the contribution guide.

@kyrofa kyrofa closed this as completed in 4e5010f Oct 7, 2016
@mattish91 mattish91 mentioned this issue Jan 18, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fix-in-beta
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kyrofa @thomascobb @MariusBluem