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

[python] add dependabot for python, add requirements.txt #5683

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
13 changes: 13 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
version: 2
updates:
- package-ecosystem: "pip"
directories:
- "/packages/http-client-python/eng/"
- "/packages/http-client-python/generator/test/azure"
- "/packages/http-client-python/generator/test/unbranded"
Copy link
Contributor

@msyyc msyyc Jan 23, 2025

Choose a reason for hiding this comment

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

Since there is also new added file generator/test/dev_requirements.txt, shall we add it here?

schedule:
interval: "weekly"
commit-message:
prefix: "pip prod"
prefix-development: "pip dev"
include: "scope"
11 changes: 11 additions & 0 deletions packages/http-client-python/eng/scripts/ci/dev_requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
pyright==1.1.391
pylint==3.2.7
tox==4.16.0
mypy==1.14.1
colorama==0.4.6
debugpy==1.8.2
pytest==8.3.2
coverage==7.6.1
black==24.8.0
ptvsd==4.3.2
types-PyYAML==6.0.12.8
12 changes: 1 addition & 11 deletions packages/http-client-python/generator/dev_requirements.txt
Original file line number Diff line number Diff line change
@@ -1,12 +1,2 @@
-e ./generator
pyright==1.1.391
pylint==3.2.7
tox==4.16.0
mypy==1.14.1
colorama==0.4.6
debugpy==1.8.2
pytest==8.3.2
coverage==7.6.1
black==24.8.0
ptvsd==4.3.2
types-PyYAML==6.0.12.8
-r ../eng/scripts/ci/dev_requirements.txt
13 changes: 13 additions & 0 deletions packages/http-client-python/generator/requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
black==24.8.0
click==8.1.8
docutils==0.21.2
Jinja2==3.1.3
m2r2==0.3.3.post2
MarkupSafe==3.0.2
mistune==0.8.4
mypy-extensions==1.0.0
packaging==24.2
pathspec==0.12.1
platformdirs==4.3.6
PyYAML==6.0.1
tomli==2.0.1
Copy link
Contributor

@msyyc msyyc Jan 22, 2025

Choose a reason for hiding this comment

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

Since these dependencies are already declared in setup.py, we removed this file in https://github.com/microsoft/typespec/pull/5445/files to reduce maintenance effort caused by that dependencies are declared in multi places. Why do we add this file requirements.txt again?

Original file line number Diff line number Diff line change
@@ -1,15 +1,7 @@
setuptools==69.5.1
-r ../dev_requirements.txt
-e ../../
aiohttp;python_full_version>="3.5.2"
requests==2.32.2
pytest
pytest-asyncio==0.14.0;python_full_version>="3.5.2"
azure-core==1.30.0
azure-mgmt-core==1.3.2
pyright==1.1.391
pylint==3.2.7
tox==4.18.1
mypy==1.14.1

# only for azure
-e ./generated/azure-client-generator-core-access
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
-r ../../eng/scripts/ci/dev_requirements.txt
aiohttp
pyright==1.1.391
pytest-asyncio==0.14.0
requests==2.32.2
Comment on lines +3 to +5
Copy link
Contributor

@msyyc msyyc Jan 23, 2025

Choose a reason for hiding this comment

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

pyright seems already be declared in ../../eng/scripts/ci/dev_requirements.txt

Original file line number Diff line number Diff line change
@@ -1,14 +1,5 @@
setuptools==69.5.1
-r ../dev_requirements.txt
-e ../../
aiohttp;python_full_version>="3.5.2"
requests==2.32.2
pytest
pytest-asyncio==0.14.0;python_full_version>="3.5.2"
pyright==1.1.391
pylint==3.2.7
tox==4.18.1
mypy==1.14.1


# common test case
-e ./generated/authentication-api-key
Expand Down
Loading