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

Consolidate with lifeonmarspt/certbot-route53 ? #2

Open
jsha opened this issue Mar 11, 2017 · 2 comments
Open

Consolidate with lifeonmarspt/certbot-route53 ? #2

jsha opened this issue Mar 11, 2017 · 2 comments

Comments

@jsha
Copy link

jsha commented Mar 11, 2017

Hi! I noticed that this project has a fork, https://github.com/lifeonmarspt/certbot-route53. That fork has more features, and reflects the current naming of the base tool (certbot vs letsencrypt). What do you think of consolidating on that fork? It would probably help reduce user confusion.

@mindstorms6
Copy link
Owner

Hey! I'm fine with that. I wrote this a while ago, looks like they finished it out. I can either pull their changes back in here, or just literally remove this entire repo and have a README point to their fork.

Open to either.

@jsha
Copy link
Author

jsha commented Mar 17, 2017

@lifeonmarspt, what do you think?

My feeling: Since https://github.com/lifeonmarspt/certbot-route53 has the more up-to-date package name, and the PyPI package, it probably makes sense to replace this repo contents and description with a pointer that fork. Also, if @lifeonmarspt agrees this is the right course, it might help users view certbot-route53 as the more authoritative version if they removed the "fork of" notation in GitHub (requires emailing support). With appropriate credit, of course. :-)

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

No branches or pull requests

2 participants