Skip to content

Upgrade to poetry 2.1.4#190

Merged
mjohanse-emr merged 4 commits intomainfrom
users/mjohanse/upgrade_poetry_2
Sep 8, 2025
Merged

Upgrade to poetry 2.1.4#190
mjohanse-emr merged 4 commits intomainfrom
users/mjohanse/upgrade_poetry_2

Conversation

@mjohanse-emr
Copy link
Copy Markdown
Contributor

@mjohanse-emr mjohanse-emr commented Sep 3, 2025

What does this Pull Request accomplish?

  • Updates pyproject.toml for nitypes to the new format used by poetry 2.1.4.
  • Updates Sphinx dependencies to fix poetry lock errors after moving to poetry 2.1.4
  • Updates the "latest deps" unit test workflows to stop using poetry lock --no-update
  • Updates docs/conf.py to use new pyproject.toml keys

Why should this Pull Request be merged?

There is a desire to move all python packages to build with poetry 2.1.4.

What testing has been done?

Unit tests, mypy, pyright, styleguide

Michael Johansen added 2 commits September 3, 2025 11:21
Signed-off-by: Michael Johansen <michael.johansen@ni.com>
Signed-off-by: Michael Johansen <michael.johansen@ni.com>
Signed-off-by: Michael Johansen <michael.johansen@ni.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 3, 2025

Test Results

    44 files  ±0      44 suites  ±0   21m 47s ⏱️ -18s
 2 190 tests ±0   2 190 ✅ ±0  0 💤 ±0  0 ❌ ±0 
49 738 runs  ±0  49 738 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit ea72de6. ± Comparison against base commit 9c07e9f.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Collaborator

@bkeryan bkeryan left a comment

Choose a reason for hiding this comment

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

Approved with suggestions (remove .tox)

Signed-off-by: Michael Johansen <michael.johansen@ni.com>
@mjohanse-emr mjohanse-emr merged commit 71b486e into main Sep 8, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants