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

Limit libraries to versions that only support Python >=3.7, add support for Python 3.11 #76

Merged
merged 17 commits into from
Aug 2, 2023
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion AUTHORS.rst
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
- Peter Bengtsson (@peterbe)
- Graham Beckley (@grahamalama)
- Mike Cooper (@mythmon)
- Will Kahn-Greene (@willkg)
- Michael Kelly (@Osmose)
- Jannis Leidel (@jezdez)
- Les Orchard (@lmorchard)
- Mathieu Leplatre (@leplatrem)
- Les Orchard (@lmorchard)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: unneeded change.

- Mathieu Pillard (@diox)
8 changes: 4 additions & 4 deletions docs/changelog.rst
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
Changelog
---------

2022.8.0 (2022-08-18)
2023.8.0
relud marked this conversation as resolved.
Show resolved Hide resolved
~~~~~~~~~~~~~~~~~~~~~

- Add support for Sanic 21 and 22, with `aioredis` 2.x
- Drop support for Sanic 20

- Add support for Django 4.1
- Drop support for Flask 0.12, 1.0, 1.1, and 2.0

- Add support for Flask 2.2
- Add support for Python 3.11

2022.7.0 (2022-07-12)
~~~~~~~~~~~~~~~~~~~~~
Expand Down
1 change: 0 additions & 1 deletion tests/requirements/sanic.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# these are constrained by the files in tests/constraints/*.txt
# to support a triple stack Django/Flask/Sanic
aiohttp
redis
Sanic
sanic_redis
sanic-testing
Expand Down
17 changes: 1 addition & 16 deletions tests/sanic/test_sanic.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,11 @@
from redis import asyncio as aioredis
from sanic import Sanic, response
from sanic_redis import SanicRedis
from sanic_testing.testing import SanicTestClient

from dockerflow import health
from dockerflow.sanic import Dockerflow, checks

if sanic.__version__.startswith("20."):
from sanic.testing import SanicTestClient
else:
from sanic_testing.testing import SanicTestClient


class FakeRedis:
def __init__(self, *args, error=None, **kw):
Expand Down Expand Up @@ -91,17 +87,6 @@ def test_client(app):
return SanicTestClient(app)


@pytest.mark.skipif(not sanic.__version__.startswith("20."), reason="requires sanic 20")
def test_instantiating_sanic_20(app):
dockerflow = Dockerflow()
assert "dockerflow.heartbeat" not in app.router.routes_names
dockerflow.init_app(app)
assert "dockerflow.heartbeat" in app.router.routes_names


@pytest.mark.skipif(
sanic.__version__.startswith("20."), reason="requires sanic 21 or later"
)
def test_instantiating(app):
Dockerflow()
assert ("__heartbeat__",) not in app.router.routes_all
Expand Down