Skip to content

ci: fix pre-release workflow to use UV_DYNAMIC_VERSIONING_BYPASS#176

Merged
Aaron ("AJ") Steers (aaronsteers) merged 1 commit into
mainfrom
devin/1782183811-fix-prerelease-versioning
Jun 23, 2026
Merged

ci: fix pre-release workflow to use UV_DYNAMIC_VERSIONING_BYPASS#176
Aaron ("AJ") Steers (aaronsteers) merged 1 commit into
mainfrom
devin/1782183811-fix-prerelease-versioning

Conversation

@aaronsteers

Copy link
Copy Markdown
Member

Summary

Follow-up to #175 (git tag-based versioning). The pre-release workflow was still using sed to edit _version.py, which is now ignored since the version comes from git tags.

-      - name: Set pre-release version
-        run: |
-          VERSION=\"${{ inputs.version }}\"
-          sed -i \"s/^__version__: str = \\\".*\\\"/__version__: str = \\\"${VERSION}\\\"/\" src/airbyte_api/_version.py
-
-      - name: Build package
-        run: uv run poe build
+      - name: Build package
+        run: uv build
+        env:
+          UV_DYNAMIC_VERSIONING_BYPASS: ${{ inputs.version }}

UV_DYNAMIC_VERSIONING_BYPASS overrides the VCS-derived version at build time — this is the same pattern PyAirbyte uses for its pre-release workflow.

Also adds fetch-depth: 0 to the checkout step for consistency.

Link to Devin session: https://app.devin.ai/sessions/854c664803f3400387fdaa02e123b888
Requested by: Aaron ("AJ") Steers (@aaronsteers)

The pre-release workflow was using sed to edit _version.py, which is now
ignored since #175 switched to git tag-based versioning. Replace with
UV_DYNAMIC_VERSIONING_BYPASS env var (matching PyAirbyte pattern).

Also adds fetch-depth: 0 to the checkout step.

Co-Authored-By: AJ Steers <aj@airbyte.io>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@aaronsteers Aaron ("AJ") Steers (aaronsteers) marked this pull request as ready for review June 23, 2026 03:04
Copilot AI review requested due to automatic review settings June 23, 2026 03:04
@aaronsteers Aaron ("AJ") Steers (aaronsteers) merged commit abdb7d7 into main Jun 23, 2026
6 of 7 checks passed
@aaronsteers Aaron ("AJ") Steers (aaronsteers) deleted the devin/1782183811-fix-prerelease-versioning branch June 23, 2026 03:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the pre-release GitHub Actions workflow to align with git tag–based dynamic versioning by bypassing the VCS-derived version at build time, rather than editing _version.py (which is now ignored for versioning).

Changes:

  • Replace the sed-based _version.py rewrite with UV_DYNAMIC_VERSIONING_BYPASS during uv build.
  • Switch the build command from uv run poe build to uv build (equivalent given poe_tasks.toml’s build task).
  • Add fetch-depth: 0 to checkout for consistency and full history availability.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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