geerlingguy / ansible-role-certbot

Ansible Role - Certbot (for Let's Encrypt)
https://galaxy.ansible.com/geerlingguy/certbot/
MIT License
781 stars 347 forks source link

Ignore error of "Enable classic snap support" #153

Closed Haelle closed 2 years ago

Haelle commented 3 years ago

On some installations origin folder does not exist and target folder already exists

stale[bot] commented 3 years ago

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

fdeh75 commented 3 years ago

+1 debian 10

stale[bot] commented 3 years ago

This issue is no longer marked for closure.

evilhamsterman commented 2 years ago

This does a lot more than just ignore errors it's also moving around tasks and handlers. Also you shouldn't just ignore errors, you should figure out why there are errors and address them. If you are running into an error on a Debian based system the issue is properly addressed in the PR #152

evilhamsterman commented 2 years ago

This is not needed the issue is resolved by PR #152 which was just merged