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

Bump Certbot and ACME dependencies with new tests added #7

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

tedski
Copy link
Contributor

@tedski tedski commented Sep 8, 2024

The outdated dependencies are blocking home-assistant/addons#3751.

This adds some testing to the Authenticator (more to come later). The tests are configured to run via pytest and the GitHub Actions have been updated to run the tests during Compatibility Testing.

This also bumps the Cerbot/ACME dependencies to 2.11.0 to keep up with the current state and unblock home-assistant/addons#3751.

This drops support for Python 3.7 as it is EOL.

@tedski tedski force-pushed the tedski/chore/testing branch 2 times, most recently from 788a182 to a3a0e43 Compare September 8, 2024 22:59
Adds testing for the Authenticator that follow the conventions of the upstream
certbot repository.  This should ensure compatibility if this was ever merged
upstream.
- Drops support for EOL Python 3.7
- Bumps Certbot/ACME to 2.11.0
- Adds testing target for Python 3.12
@tedski tedski force-pushed the tedski/chore/testing branch from a3a0e43 to 39df441 Compare September 8, 2024 23:01
@tedski tedski marked this pull request as ready for review September 8, 2024 23:02
@tedski
Copy link
Contributor Author

tedski commented Sep 12, 2024

@stephenjust @dalbothek can you take a look at this and help unblock projects that depend on this library? Thanks!

@dalbothek
Copy link
Contributor

Thank you very much for your contribution! I might not find time to review your changes in detail until next Tuesday, but I will make sure to get a new version out by then.

@tedski
Copy link
Contributor Author

tedski commented Sep 24, 2024

Any chance this can get reviewed this week, @dalbothek ?

@dalbothek dalbothek merged commit 4f6b710 into inventage:master Sep 25, 2024
5 of 6 checks passed
@dalbothek
Copy link
Contributor

Thank you for the reminder, the changes have now been merged and a new version 0.7.0 is available.

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.

2 participants