Skip to content

Conversation

@mikeprosserni
Copy link
Collaborator

What does this Pull Request accomplish?

Updates the repo to use Poetry v2.1.4

Why should this Pull Request be merged?

image

What testing has been done?

Ran the autotests and did some interactive testing with the all_types example.

@mikeprosserni mikeprosserni requested a review from csjall as a code owner August 25, 2025 18:20
@github-actions
Copy link
Contributor

github-actions bot commented Aug 25, 2025

Test Results

   10 files  ±0     10 suites  ±0   30s ⏱️ -4s
  251 tests ±0    251 ✅ ±0  0 💤 ±0  0 ❌ ±0 
2 460 runs  ±0  2 460 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 7840c23. ± Comparison against base commit 2e3ec2a.

♻️ This comment has been updated with latest results.

@mikeprosserni mikeprosserni merged commit 7ed53fa into main Aug 25, 2025
14 checks passed
@mikeprosserni mikeprosserni deleted the users/mprosser/update-poetry branch August 25, 2025 19:02
{version = "1.49.1", python = ">=3.9,<3.12"},
{version = "1.59.0", python = ">=3.12,<3.13"},
{version = "1.67.0", python = "^3.13"},
[project.optional-dependencies]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Converting dependency groups to extras is completely wrong. Please revert this.

Before:

Metadata-Version: 2.3
Name: nipanel
Version: 0.1.0
Summary: NI Panel Python API
Author: NI
Author-email: opensource@ni.com
Requires-Python: >=3.9, !=2.7.*, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*, !=3.5.*, !=3.6.*, !=3.7.*, !=3.8.*
Classifier: Programming Language :: Python :: 3
Classifier: Programming Language :: Python :: 3.9
Classifier: Programming Language :: Python :: 3.10
Classifier: Programming Language :: Python :: 3.11
Classifier: Programming Language :: Python :: 3.12
Classifier: Programming Language :: Python :: 3.13
Requires-Dist: debugpy (>=1.8.1)
Requires-Dist: grpcio (>=1.49.0,<2.0)
Requires-Dist: ni-measurement-plugin-sdk (>=2.4.0dev1)
Requires-Dist: ni-panels-v1-proto (>=0.1.0dev1)
Requires-Dist: ni-protobuf-types (>=0.1.0dev2)
Requires-Dist: nitypes (>=0.1.0dev8)
Requires-Dist: numpy (>=1.22)
Requires-Dist: protobuf (>=4.21)
Requires-Dist: streamlit (>=1.24)
Requires-Dist: typing-extensions (>=4.13.2)
Description-Content-Type: text/markdown

# nipanel-python
Source for the nipanel python package

After:

Metadata-Version: 2.3
Name: nipanel
Version: 0.1.0
Summary: NI Panel Python API
Author: NI
Author-email: opensource@ni.com
Requires-Python: >=3.9,<4.0,!=3.9.7
Classifier: Programming Language :: Python :: 3
Classifier: Programming Language :: Python :: 3.9
Classifier: Programming Language :: Python :: 3.10
Classifier: Programming Language :: Python :: 3.11
Classifier: Programming Language :: Python :: 3.12
Classifier: Programming Language :: Python :: 3.13
Provides-Extra: dev
Provides-Extra: docs
Provides-Extra: examples
Provides-Extra: lint
Provides-Extra: test
Requires-Dist: Sphinx (>=8.2) ; (python_version >= "3.11") and (extra == "docs")
Requires-Dist: bandit[toml] (>=1.7) ; extra == "lint"
Requires-Dist: debugpy (>=1.8.1)
Requires-Dist: extra-streamlit-components (>=0.1.80) ; extra == "examples"
Requires-Dist: grpcio (>=1.49.0,<2.0)
Requires-Dist: hightime (>=0.2.2) ; extra == "test"
Requires-Dist: m2r2 (>=0.3.2) ; extra == "docs"
Requires-Dist: mypy (>=1.0) ; extra == "lint"
Requires-Dist: ni-measurement-plugin-sdk (>=2.4.0dev1)
Requires-Dist: ni-panels-v1-proto (>=0.1.0dev1)
Requires-Dist: ni-protobuf-types (>=0.1.0dev3)
Requires-Dist: ni-python-styleguide (>=0.4.1) ; extra == "lint"
Requires-Dist: nidaqmx (>=0.8.0) ; extra == "examples"
Requires-Dist: niscope (>=1.4.9) ; extra == "examples"
Requires-Dist: nitypes (>=0.1.0dev8)
Requires-Dist: numpy (>=1.22)
Requires-Dist: numpy (>=1.22) ; (python_version >= "3.9" and python_version < "3.12") and (extra == "dev")
Requires-Dist: numpy (>=1.26) ; (python_version == "3.12") and (extra == "dev")
Requires-Dist: numpy (>=2.1) ; (python_version >= "3.13") and (extra == "dev")
Requires-Dist: protobuf (>=4.21)
Requires-Dist: pyright[nodejs] (>=1.1.400) ; extra == "lint"
Requires-Dist: pytest (>=7.2) ; extra == "test"
Requires-Dist: pytest-cov (>=4.0) ; extra == "test"
Requires-Dist: pytest-mock (>=3.0) ; extra == "test"
Requires-Dist: sphinx-autoapi (>=1.8.4) ; extra == "docs"
Requires-Dist: sphinx-rtd-theme (>=1.0.0) ; extra == "docs"
Requires-Dist: streamlit (>=1.24)
Requires-Dist: streamlit-echarts (>=0.4.0) ; extra == "examples"
Requires-Dist: toml (>=0.10.2) ; extra == "docs"
Requires-Dist: types-grpcio (>=1.0) ; extra == "dev"
Requires-Dist: types-protobuf (>=4.21) ; extra == "dev"
Requires-Dist: typing-extensions (>=4.13.2)
Description-Content-Type: text/markdown

# nipanel-python
Source for the nipanel python package

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.

4 participants