Skip to content

fix: use correct Python version in release workflow#67

Merged
micmarty-deepsense merged 6 commits intomainfrom
fix/release-workflow-python-version
Mar 3, 2026
Merged

fix: use correct Python version in release workflow#67
micmarty-deepsense merged 6 commits intomainfrom
fix/release-workflow-python-version

Conversation

@micmarty-deepsense
Copy link
Contributor

Problem

The release workflow was using ${{ matrix.python-version }} but there's no matrix in this job. setup-uv fell back to the latest Python (3.14), which caused pydantic-core to fail building from source — pyo3 0.24.1 only supports up to Python 3.13.

The PYTHON_VERSION: "3.10" env var was already defined but never referenced.

Fix

  • python-version: ${{ matrix.python-version }}${{ env.PYTHON_VERSION }}
  • Remove make install-dependencies — not needed in the release job (only uv build + publish)

Next step

After merging, delete and re-create the 0.0.42 GitHub release to re-trigger this workflow.

micmarty-deepsense and others added 6 commits January 27, 2026 13:21
- Filter "none" from OTEL_TRACES_EXPORTER and OTEL_METRICS_EXPORTER
- Return None from get_trace_provider/get_metric_provider when no exporters
- Prevents NotImplementedError on plugin startup with OTEL disabled
- Bump version 0.0.41 → 0.0.42

Fixes plugin crash when OTEL_TRACES_EXPORTER=none or OTEL_METRICS_EXPORTER=none
Covers get_settings() filtering, provider None returns, and the
wrap_in_fastapi no-crash regression for OTEL_TRACES_EXPORTER=none
and OTEL_METRICS_EXPORTER=none.
Replace `matrix.python-version` (undefined in this job) with
`env.PYTHON_VERSION` so the release uses Python 3.10 as intended.
Also drop the unnecessary `make install-dependencies` step — the
release job only needs `uv build` and the publish action.
@micmarty-deepsense micmarty-deepsense merged commit cd81430 into main Mar 3, 2026
14 checks passed
@micmarty-deepsense micmarty-deepsense deleted the fix/release-workflow-python-version branch March 3, 2026 09:48
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.

2 participants