Skip to content

fix: align pyproject.toml version directly in generation workflows#392

Merged
louis-sanna-dev merged 1 commit intomainfrom
fix/align-version-in-generation-workflow
Feb 27, 2026
Merged

fix: align pyproject.toml version directly in generation workflows#392
louis-sanna-dev merged 1 commit intomainfrom
fix/align-version-in-generation-workflow

Conversation

@louis-sanna-dev
Copy link
Contributor

Summary

Embeds the version alignment logic directly in each SDK generation workflow instead of using a separate workflow_run trigger.

Why the previous approaches failed:

  • pull_request trigger: GITHUB_TOKEN actions don't trigger PR workflows (security measure)
  • workflow_run trigger: Runs on the triggering workflow's branch (main), not the PR branch

This approach works because:

  • Same workflow = same GITHUB_TOKEN permissions
  • needs: generate ensures it runs after the PR is created
  • No trigger chain or branch detection issues

Changes

  • Added align-version job to all 3 SDK generation workflows
  • Each job finds the most recent PR from github-actions bot and updates its pyproject.toml
  • Deleted the standalone align_pyproject_version.yaml workflow

Test plan

  1. Merge this PR
  2. Trigger "Generate MISTRALAI" workflow
  3. Verify the align-version job runs after generation and updates pyproject.toml in the PR

Instead of using a separate workflow with workflow_run trigger (which
runs on the wrong branch), embed the version alignment logic directly
in each SDK generation workflow as a job that runs after generation.

This approach works because:
- Same workflow = same GITHUB_TOKEN permissions
- needs: generate ensures it runs after PR is created
- No trigger gymnastics or branch detection issues

Deletes the standalone align_pyproject_version.yaml workflow.
@louis-sanna-dev louis-sanna-dev merged commit 176b844 into main Feb 27, 2026
9 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.

2 participants