Skip to content

Commit

Permalink
Bump FAST API upper bounds (#1634)
Browse files Browse the repository at this point in the history
* create pydantic flowchart classes

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fixing pydantic class conversions

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* minor modifications and code comments

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix pytests for flowchart and managers

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix pytests

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix lint errors and pytests

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix lint issues and merge main

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix initialization issues and lint issues

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix push issue

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* revert class method to static method

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* update method comments

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* revert back method shuffles for better PR reviews

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* addressing PR comments

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* not-working version of pydantic shift

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* non-working p2

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* not-working v3

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* modify metadata classes and update pytests

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix all pytests

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix pytest for coverage

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* address PR comments1

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix lint issues

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* add hash for pylint

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix lint errors

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix lint errors

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* upgrade fastapi and test

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* upgrade pydantic to accomodate fastapi

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* pin lowerbound pydantic version

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* lint fix

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* add release note

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* address PR comments

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* modify release note

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

---------

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>
Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>
ravi-kumar-pilla and rashidakanchwala authored Nov 16, 2023
1 parent 0025e71 commit 7fd5971
Showing 4 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions RELEASE.md
Original file line number Diff line number Diff line change
@@ -15,6 +15,7 @@ Please follow the established format:
- Ensure Kedro-Viz works when hosted on a URL subpath. (#1621)
- Fix shareable URL modal to appear across the app. (#1639)
- Refactor flowchart dataclasses to pydantic base models. (#1565)
- Bump FAST API upper bounds. (#1634)

# Release 6.6.1

1 change: 1 addition & 0 deletions package/features/steps/lower_requirements.txt
Original file line number Diff line number Diff line change
@@ -13,3 +13,4 @@ strawberry-graphql==0.192.0
networkx==2.5
orjson==3.9
secure==0.3.0
pydantic==1.10
4 changes: 3 additions & 1 deletion package/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
packaging~=23.0
kedro>=0.17.5
ipython>=7.0.0, <9.0
fastapi>=0.73.0, <0.96.0
fastapi>=0.73.0,<0.200.0; python_version >= '3.8'
fastapi>=0.73.0,<0.104.0; python_version < '3.8'
pydantic<2
fsspec[s3]>=2021.4, <2024.1
aiofiles>=22.1.0
uvicorn[standard]~=0.22.0
2 changes: 1 addition & 1 deletion package/test_requirements.txt
Original file line number Diff line number Diff line change
@@ -8,7 +8,6 @@ behave~=1.2
black~=23.3
boto3~=1.26
flake8~=5.0
fastapi[all]>=0.73.0, <0.96.0
isort~=5.11
matplotlib~=3.5
mypy~=1.0
@@ -26,6 +25,7 @@ sqlalchemy-stubs~=0.4
strawberry-graphql[cli]>=0.99.0, <1.0
trufflehog~=2.2
typing_extensions~=4.7.0; python_version < '3.9'
httpx~=0.24.0

# mypy
types-aiofiles==0.1.3

0 comments on commit 7fd5971

Please sign in to comment.