#253: Made dependencies optional in file `pyproject.toml` #254

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

Jump to bottom

Merged

ckunki merged 27 commits into main from refactoring/253-Made_dependencies_optional_in_file_pyproject.toml

Sep 11, 2025

Contributor

ckunki commented Sep 8, 2025

Closes #253


          253: Made dependencies optional in file pyproject.toml

a3479ba

ckunki had a problem deploying to manual-approval

September 8, 2025 11:21

— with

GitHub Actions Error

ckunki had a problem deploying to manual-approval

September 8, 2025 11:24

— with

GitHub Actions Failure

ckunki had a problem deploying to manual-approval

September 8, 2025 11:29

— with

GitHub Actions Failure

ckunki added 2 commits

September 8, 2025 13:30


          Removed section dependendencies

a3573c4


          Updated optional dependencies

20cf94e

fixed syntax in file pyproject.toml, added user guide, re-locked dependencies

ckunki had a problem deploying to manual-approval

September 8, 2025 13:27

— with

GitHub Actions Failure

ckunki had a problem deploying to manual-approval

September 8, 2025 13:50

— with

GitHub Actions Failure


          Install all optional dependencies also for lint

a79ed7a

Made all dependencies optional again

ckunki commented

View reviewed changes

.github/workflows/checks.yml

    
                      with:

                        python-version: ${{ matrix.python-version }}

                    - name: Install All Optional Dependencies (aka. "extras")

Contributor Author

ckunki Sep 8, 2025

PTB currently only offers action python-environment to be called with a list of space-separated extras.
Proposal: Add support for requesting to install "all extras".

ckunki had a problem deploying to manual-approval

September 8, 2025 14:39

— with

GitHub Actions Failure


          Removed commented-out lines

a910215

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated

    
              ibis-framework = [ "ibis-framework [exasol] >=9.1.0, <10" ]

              GitPython = [ "GitPython >=2.1.0" ]

              yaspin = [ "yaspin >=3.1.0, <4" ]

              exasol-text-ai-extension = [ "exasol-text-ai-extension >=0.1.0, <2.0.0" ]

Collaborator

tkilias Sep 9, 2025

its own category

Collaborator

tkilias Sep 9, 2025

Before we can do this, we need to move exasol.ai.text.impl.utils.transformers_utils.download_transformers_model to transformers extension. For the moment, we put it together with transformer-extensions into one category.

Contributor Author

ckunki Sep 9, 2025

See tickets

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

tkilias reviewed

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved


          made some dependencies mandatory

added dependency for win32
grouped optional dependencies into categories

ckunki had a problem deploying to manual-approval

September 9, 2025 10:00

— with

GitHub Actions Failure

ckunki had a problem deploying to manual-approval

September 11, 2025 07:26

— with

GitHub Actions Failure


          Removed old table with column pypi

c51cb17

tkilias previously approved these changes

View reviewed changes

ckunki dismissed tkilias’s stale review via

a6cd96d

September 11, 2025 07:59

ckunki temporarily deployed to manual-approval

September 11, 2025 07:59

— with

GitHub Actions Inactive

tkilias previously approved these changes

View reviewed changes


          relocked dependencies

a6cd96d

ckunki temporarily deployed to text-ai-prerelease

September 11, 2025 08:48

— with

GitHub Actions Inactive


          Added optional dependency for integration tests with external ITDE

14d8923

ckunki dismissed tkilias’s stale review via

14d8923

September 11, 2025 09:15

ckunki had a problem deploying to manual-approval

September 11, 2025 09:15

— with

GitHub Actions Failure

ckunki had a problem deploying to manual-approval

September 11, 2025 09:17

— with

GitHub Actions Failure


          Run ordinary itests without confirmation

15affba

ckunki had a problem deploying to manual-approval

September 11, 2025 09:22

— with

GitHub Actions Failure


          Added optional dependency for integration tests with external ITDE (2)

9df20ba

ckunki had a problem deploying to manual-approval

September 11, 2025 09:34

— with

GitHub Actions Failure


          Added optional dependency for itests with external ITDE (3)

01c5e25

ckunki had a problem deploying to manual-approval

September 11, 2025 09:47

— with

GitHub Actions Failure


          Added optional dependency for itests with external ITDE (3)

fe9f021

Added optional dependency bucketfs

ckunki had a problem deploying to manual-approval

September 11, 2025 10:01

— with

GitHub Actions Failure


          Added optional dependency for itests with external ITDE (3)

a448166

Added optional dependency ibis

ckunki had a problem deploying to manual-approval

September 11, 2025 11:02

— with

GitHub Actions Failure


          Added optional dependency for itests with external ITDE (3)

9533edd

Added optional dependency sqlalchemy

ckunki temporarily deployed to manual-approval

September 11, 2025 11:29

— with

GitHub Actions Inactive


          Re-enabled precondition slow test approval for ordinary-itests

d3cdead

ckunki temporarily deployed to text-ai-prerelease

September 11, 2025 11:29

— with

GitHub Actions Inactive

tkilias approved these changes

View reviewed changes

sonarqubecloud bot commented Sep 11, 2025

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

ckunki merged commit bcd0ce3 into main

26 checks passed

ckunki deleted the refactoring/253-Made_dependencies_optional_in_file_pyproject.toml branch

September 11, 2025 12:02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet