Skip to content

client: remove unused dependencies from build configuration #430

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.

Sign up for GitHub

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

Open
dogukancagatay wants to merge 2 commits into
base: main
Choose a base branch
from
+1 −10
Open

client: remove unused dependencies from build configuration #430

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8a350c9
client: remove unused dependencies from build configuration
dogukancagatay Oct 30, 2025
cbed6a3
client: add ruff to fedora dependencies
dogukancagatay Oct 30, 2025
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
2 changes: 1 addition & 1 deletion Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ build-dep-fedora:
python3-certifi \
python3-hatch-vcs \
python3-hatchling \
python3-PyMySQL \
python3-wheel \
python3-flake8 \
python3-isort \
Expand All @@ -63,6 +62,7 @@ build-dep-fedora:
python3-types-requests \
python3-setuptools_scm \
rpmdevtools \
ruff \
tar

rpm:
Expand Down
8 changes: 0 additions & 8 deletions aiven-client.spec
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,7 @@ Summary: Aiven Client
License: ASL 2.0
BuildArch: noarch
BuildRequires: python3-devel
BuildRequires: python3dist(wheel)
BuildRequires: python3dist(hatchling)
BuildRequires: python3dist(hatch-vcs)
BuildRequires: python3dist(flake8)
BuildRequires: python3dist(mypy)
BuildRequires: python3dist(pytest)
Copy link
Contributor Author

@dogukancagatay dogukancagatay Nov 3, 2025

Choose a reason for hiding this comment

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

dev group dependencies could be included to be generated dynamically, but pytest is the only dev dependency used.

BuildRequires: python3dist(requests)
BuildRequires: python3dist(pymysql)
BuildRequires: python3dist(certifi)


%description
Expand Down
1 change: 0 additions & 1 deletion pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ dev = [
"pytest",
# Lock isort to the same major version used in downstream build environments
"isort>=6.1.0,<7.0",
"click",
"ruff",
]

Expand Down