feat: Rev minimum python version to 3.10 and add python 3.14 (#168)

Co-authored-by: Paul Spangler <[email protected]>

Author: rbell517

.flake8

@@ -4,11 +4,21 @@ max-line-length = 119
 # but keep line length to 88 in general
 # - use "black" to auto-format
 
+# D400 and D205 are because we sometimes split the first line of the docstring
+# F821 is temporary, until code is complete and stuff like ObjectDisposedException is gone
 ignore =
-    # D400 and D205 are because we sometimes split the first line of the docstring
-    *.py E203, W503, D100,D101,D102,D105,D106,D107, D400,D205, D415
-    # temporary, until code is complete and stuff like ObjectDisposedException is gone:
-    *.py F821
+    E203,
+    W503,
+    D100,
+    D101,
+    D102,
+    D105,
+    D106,
+    D107,
+    D400,
+    D205,
+    D415,
+    F821
 
 # Config for flake8-docstrings
 docstring-convention = google

.github/workflows/python-package.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ['3.9', '3.10', '3.11', '3.12', '3.13']
+        python-version: ['3.10', '3.11', '3.12', '3.13', '3.14']
     steps:
       - uses: actions/checkout@v4
       - name: Install poetry
@@ -29,7 +29,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ['3.9', '3.10', '3.11', '3.12', '3.13']
+        python-version: ['3.10', '3.11', '3.12', '3.13', '3.14']
       # Do not run in parallel to limit parallel integration tests stomping on each other
       max-parallel: 1
     steps:
@@ -59,7 +59,7 @@ jobs:
         run: pipx install poetry
       - uses: actions/setup-python@v4
         with:
-          python-version: 3.9
+          python-version: 3.10
           cache: "poetry"
       - run: poetry install
       - name: Semantic release

.readthedocs.yaml

@@ -7,9 +7,9 @@ version: 2
 
 # Set the version of Python and other tools you might need
 build:
-  os: ubuntu-20.04
+  os: ubuntu-lts-latest
   tools:
-    python: "3.9"
+    python: "3.10"
 
 # Build documentation in the docs/ directory with Sphinx
 sphinx:

CONTRIBUTING.md

@@ -37,7 +37,7 @@ To contribute to this project, it is recommended that you follow these steps:
 
 ## Testing
 
-Before running any tests, you must have a supported version of Python (3.9+) and [Poetry](https://python-poetry.org/docs/) installed locally.
+Before running any tests, you must have a supported version of Python (3.10+) and [Poetry](https://python-poetry.org/docs/) installed locally.
 
 It is also helpful to install SystemLink Server and configure the NI Web Server
 to run on localhost.

README.rst

@@ -18,7 +18,7 @@ Requirements
 **nisystemlink-clients** has the following requirements:
 
 * Access to a SystemLink Server or SystemLink Enterprise installation
-* CPython 3.9+
+* CPython 3.10+
 
 .. _installation_section:
 

nisystemlink/clients/artifact/_artifact_client.py

@@ -1,6 +1,6 @@
 """Implementation of ArtifactClient"""
 
-from typing import BinaryIO, Optional
+from typing import BinaryIO
 
 from nisystemlink.clients import core
 from nisystemlink.clients.core._uplink._base_client import BaseClient
@@ -18,7 +18,7 @@
 
 
 class ArtifactClient(BaseClient):
-    def __init__(self, configuration: Optional[core.HttpConfiguration] = None):
+    def __init__(self, configuration: core.HttpConfiguration | None = None):
         """Initialize an instance.
 
         Args:

nisystemlink/clients/assetmanagement/_asset_management_client.py

@@ -1,6 +1,6 @@
 """Implementation of AssetManagementClient."""
 
-from typing import List, Optional
+from typing import List
 
 from nisystemlink.clients import core
 from nisystemlink.clients.core._http_configuration import HttpConfiguration
@@ -17,7 +17,7 @@
     on_exception=retry.CONNECTION_ERROR,
 )
 class AssetManagementClient(BaseClient):
-    def __init__(self, configuration: Optional[HttpConfiguration] = None):
+    def __init__(self, configuration: HttpConfiguration | None = None):
         """Initialize an instance.
 
         Args:
@@ -126,7 +126,7 @@ def delete_assets(self, ids: List[str]) -> models.DeleteAssetsResponse:
     @post("assets/{assetId}/file", args=[Path("assetId"), Field("fileIds")])
     def link_files(
         self, asset_id: str, file_ids: List[str]
-    ) -> Optional[models.LinkFilesPartialSuccessResponse]:
+    ) -> models.LinkFilesPartialSuccessResponse | None:
         """Link files to an asset.
 
         Args:

nisystemlink/clients/assetmanagement/models/_asset.py

@@ -1,5 +1,5 @@
 from datetime import datetime
-from typing import Dict, List, Optional
+from typing import Dict, List
 
 from nisystemlink.clients.core._uplink._json_model import JsonModel
 from pydantic import Field
@@ -17,98 +17,98 @@
 class Asset(JsonModel):
     """Model for an object describing an asset with all of its properties."""
 
-    model_name: Optional[str] = None
+    model_name: str | None = None
     """Gets or sets model name of the asset."""
 
-    model_number: Optional[int] = None
+    model_number: int | None = None
     """Gets or sets model number of the asset."""
 
-    serial_number: Optional[str] = None
+    serial_number: str | None = None
     """Gets or sets serial number of the asset."""
 
-    vendor_name: Optional[str] = None
+    vendor_name: str | None = None
     """Gets or sets vendor name of the asset."""
 
-    vendor_number: Optional[int] = None
+    vendor_number: int | None = None
     """Gets or sets vendor number of the asset."""
 
-    bus_type: Optional[AssetBusType] = None
+    bus_type: AssetBusType | None = None
     """Gets or sets all supported bus types for an asset."""
 
-    name: Optional[str] = None
+    name: str | None = None
     """Gets or sets name of the asset."""
 
-    asset_type: Optional[AssetType] = None
+    asset_type: AssetType | None = None
     """Gets or sets all supported asset types."""
 
-    discovery_type: Optional[AssetDiscoveryType] = None
+    discovery_type: AssetDiscoveryType | None = None
     """Gets or sets the discovery type."""
 
-    firmware_version: Optional[str] = None
+    firmware_version: str | None = None
     """Gets or sets firmware version of the asset."""
 
-    hardware_version: Optional[str] = None
+    hardware_version: str | None = None
     """Gets or sets hardware version of the asset."""
 
-    visa_resource_name: Optional[str] = None
+    visa_resource_name: str | None = None
     """Gets or sets VISA resource name of the asset."""
 
-    temperature_sensors: Optional[List[TemperatureSensor]] = None
+    temperature_sensors: List[TemperatureSensor] | None = None
     """Gets or sets an array of temperature sensor information."""
 
-    supports_self_calibration: Optional[bool] = None
+    supports_self_calibration: bool | None = None
     """Gets or sets whether the asset supports self-calibration."""
 
-    supports_external_calibration: Optional[bool] = None
+    supports_external_calibration: bool | None = None
     """Gets or sets whether the asset supports external calibration."""
 
-    custom_calibration_interval: Optional[int] = None
+    custom_calibration_interval: int | None = None
     """Gets or sets the interval represented in months used for computing calibration due date."""
 
-    self_calibration: Optional[SelfCalibration] = None
+    self_calibration: SelfCalibration | None = None
     """Gets or sets the last self-calibration of the asset."""
 
-    is_NI_asset: Optional[bool] = Field(alias="isNIAsset", default=None)
+    is_NI_asset: bool | None = Field(alias="isNIAsset", default=None)
     """Gets or sets whether this asset is an NI asset (true) or a third-party asset (false)."""
 
-    id: Optional[str] = None
+    id: str | None = None
     """Gets or sets unique identifier of the asset."""
 
-    location: Optional[AssetLocation] = None
+    location: AssetLocation | None = None
     """Model for information about the asset location, presence and the connection status of the system"""
 
-    calibration_status: Optional[CalibrationStatus] = None
+    calibration_status: CalibrationStatus | None = None
     """Gets or sets the calibration category the asset belongs to based on the next due calibration date."""
 
-    is_system_controller: Optional[bool] = None
+    is_system_controller: bool | None = None
     """Gets or sets whether this asset represents a System Controller."""
 
-    external_calibration: Optional[ExternalCalibration] = None
+    external_calibration: ExternalCalibration | None = None
     """Gets or sets the last external calibration of the asset."""
 
-    workspace: Optional[str] = None
+    workspace: str | None = None
     """Gets or sets the ID of the workspace."""
 
-    properties: Optional[Dict[str, str]] = None
+    properties: Dict[str, str] | None = None
     """	Gets or sets key-value-pair metadata associated with an asset."""
 
-    keywords: Optional[List[str]] = None
+    keywords: List[str] | None = None
     """Gets or sets words or phrases associated with an asset."""
 
-    last_updated_timestamp: Optional[datetime] = None
+    last_updated_timestamp: datetime | None = None
     """Gets or sets ISO-8601 formatted timestamp specifying the last date that the asset has had a property update."""
 
-    file_ids: Optional[List[str]] = None
+    file_ids: List[str] | None = None
     """Gets or sets all files linked to the asset."""
 
-    supports_self_test: Optional[bool] = None
+    supports_self_test: bool | None = None
     """Gets or sets whether the asset supports self-test."""
 
-    supports_reset: Optional[bool] = None
+    supports_reset: bool | None = None
     """Gets or sets whether the asset supports reset."""
 
-    part_number: Optional[str] = None
+    part_number: str | None = None
     """Gets or sets part number of the asset."""
 
-    out_for_calibration: Optional[bool] = None
+    out_for_calibration: bool | None = None
     """Get or set whether the asset is out for calibration."""

nisystemlink/clients/assetmanagement/models/_asset_calibration.py

@@ -2,7 +2,7 @@
 
 from datetime import datetime
 from enum import Enum
-from typing import List, Optional
+from typing import List
 
 from nisystemlink.clients.core._uplink._json_model import JsonModel
 
@@ -19,18 +19,18 @@ class CalibrationStatus(Enum):
 class TemperatureSensor(JsonModel):
     """Temperature sensor information."""
 
-    name: Optional[str] = None
+    name: str | None = None
     """Gets or sets sensor name."""
 
     reading: float
     """Gets or sets sensor reading."""
 
 
 class SelfCalibration(JsonModel):
-    temperature_sensors: Optional[List[TemperatureSensor]] = None
+    temperature_sensors: List[TemperatureSensor] | None = None
     """Gets or sets an array of temperature sensor information."""
 
-    is_limited: Optional[bool] = None
+    is_limited: bool | None = None
     """Gets or sets whether the last self-calibration of the asset was a limited calibration."""
 
     date: datetime
@@ -45,10 +45,10 @@ class CalibrationMode(Enum):
 
 
 class ExternalCalibration(JsonModel):
-    temperature_sensors: Optional[List[TemperatureSensor]] = None
+    temperature_sensors: List[TemperatureSensor] | None = None
     """Gets or sets an array of temperature sensor information."""
 
-    is_limited: Optional[bool] = None
+    is_limited: bool | None = None
     """Gets or sets whether the last external calibration of the asset was a limited calibration."""
 
     date: datetime
@@ -60,14 +60,14 @@ class ExternalCalibration(JsonModel):
     next_recommended_date: datetime
     """Gets or sets ISO-8601 formatted timestamp specifying the recommended date for the next external calibration."""
 
-    next_custom_due_date: Optional[datetime] = None
+    next_custom_due_date: datetime | None = None
     """Gets or sets ISO-8601 formatted timestamp specifying the date for the next external calibration."""
 
-    resolved_due_date: Optional[datetime] = None
+    resolved_due_date: datetime | None = None
     """Gets ISO-8601 formatted timestamp specifying the resolved due date for external calibration."""
 
-    comments: Optional[str] = None
+    comments: str | None = None
     """Gets or sets calibration comments provided by an operator."""
 
-    entry_type: Optional[CalibrationMode] = None
+    entry_type: CalibrationMode | None = None
     """Gets or sets whether automatically discovered the calibration data for an asset or manually entered."""

nisystemlink/clients/assetmanagement/models/_asset_location.py

@@ -1,5 +1,4 @@
 from enum import Enum
-from typing import Optional
 
 from nisystemlink.clients.core._uplink._json_model import JsonModel
 
@@ -36,7 +35,7 @@ class AssetPresenceWithSystemConnection(JsonModel):
     asset_presence: AssetPresenceStatus
     """Gets or sets the status of an asset's presence in a system."""
 
-    system_connection: Optional[SystemConnection] = None
+    system_connection: SystemConnection | None = None
     """Gets or sets whether or not the minion is connected to the server and has updated the server with its data."""
 
 
@@ -50,19 +49,19 @@ class AssetPresence(JsonModel):
 class _AssetLocation(JsonModel):
     """local model for information about the asset location, presence and the connection status of the system."""
 
-    minion_id: Optional[str] = None
+    minion_id: str | None = None
     """Gets or sets identifier of the minion where the asset is located."""
 
-    physical_location: Optional[str] = None
+    physical_location: str | None = None
     """Gets or sets the physical location of the asset."""
 
-    parent: Optional[str] = None
+    parent: str | None = None
     """Gets or sets the parent of the asset."""
 
-    resource_uri: Optional[str] = None
+    resource_uri: str | None = None
     """Gets or sets identifier of a resource."""
 
-    slot_number: Optional[int] = None
+    slot_number: int | None = None
     """Gets or sets the number of the slot in which the asset is located."""