-
Notifications
You must be signed in to change notification settings - Fork 3k
Drop python 3.8 and 3.9 #9140
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
Drop python 3.8 and 3.9 #9140
Changes from all commits
6e92eb2
57b404d
0610012
7b81e54
3f3e56f
a9571bc
8bace97
4c9ca4b
8f85237
b88b7e2
10667e5
647133d
7807018
93d8bab
2dbe968
dcb3f47
6175a2c
0d90abc
314d5cb
74ec4aa
5ec5210
57faed7
d3eb717
1e66e29
8649c43
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
--- | ||
"gradio": minor | ||
"gradio_client": minor | ||
--- | ||
|
||
feat:Drop python 3.8 and 3.9 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -49,6 +49,8 @@ jobs: | |
always_install_pnpm: true | ||
build_lite: true | ||
skip_build: true | ||
python_version: "3.10" | ||
test: true | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @pngwn I added |
||
- run: pnpm exec playwright install chromium firefox | ||
- name: Run Lite E2E tests | ||
run: | | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,6 @@ | ||
from typing import List, Optional | ||
from typing import Annotated, Optional | ||
|
||
from typer import Option | ||
from typing_extensions import Annotated | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Many (but sadly not all) classes from |
||
from gradio_client import Client | ||
|
||
|
@@ -17,7 +16,7 @@ def main( | |
str, Option(help="Discord bot token. Get one on the discord website.") | ||
] = None, | ||
api_names: Annotated[ | ||
List[str], Option(help="Api names to turn into discord bots") | ||
list[str], Option(help="Api names to turn into discord bots") | ||
] = None, | ||
to_id: Annotated[ | ||
Optional[str], Option(help="Name of the space used to host the discord bot") | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,7 +7,7 @@ name = "gradio_client" | |
dynamic = ["version", "dependencies", "readme"] | ||
description = "Python library for easily interacting with trained machine learning models" | ||
license = "Apache-2.0" | ||
requires-python = ">=3.8" | ||
requires-python = ">=3.10" | ||
authors = [ | ||
{ name = "Abubakar Abid", email = "[email protected]" }, | ||
{ name = "Ali Abid", email = "[email protected]" }, | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have to manually specify this b/c our CI uses the version of the action that is on
main
, and the one currently onmain
is using 3.8