Skip to content

ci: remove [skip ci] from uv.lock workflow to fix release publishing#1046

Merged
edwinjosechittilappilly merged 1 commit into
mainfrom
fix-skip-ci
Feb 28, 2026
Merged

ci: remove [skip ci] from uv.lock workflow to fix release publishing#1046
edwinjosechittilappilly merged 1 commit into
mainfrom
fix-skip-ci

Conversation

@edwinjosechittilappilly

@edwinjosechittilappilly edwinjosechittilappilly commented Feb 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Removes [skip ci] from the automated commit message in the update-uv-lock.yml workflow

Problem

The release 0.2.5 merge to main (#1021) did not trigger the MCP, Python SDK, or TypeScript SDK publish workflows, even though the commit modified all three trigger paths (sdks/mcp/pyproject.toml, sdks/python/pyproject.toml, sdks/typescript/package.json).

Root cause: When PR #1021 was squash-merged, GitHub concatenated all intermediate commit messages into the squash commit body. One of those was the automated uv.lock update commit:

chore: update uv.lock files after version bump [skip ci]

GitHub Actions scans the entire commit message (title + body) for [skip ci]. Finding it in the body caused GitHub to skip all workflows for that push — including the three publish workflows.

Why this fix is safe

The [skip ci] was originally added to prevent the uv.lock commit from re-triggering the update-uv-lock workflow in a loop. However, this is unnecessary because the workflow already uses a paths filter that only matches pyproject.toml files:

paths:
  - 'pyproject.toml'
  - 'sdks/python/pyproject.toml'
  - 'sdks/mcp/pyproject.toml'

A commit that only modifies uv.lock files will never match this filter, so no infinite loop is possible.

Update .github/workflows/update-uv-lock.yml to remove the "[skip ci]" token from the automated commit message that updates uv.lock files. This change ensures CI will run on commits produced by the update-uv-lock workflow (uv.lock, sdks/python/uv.lock, sdks/mcp/uv.lock).
Copilot AI review requested due to automatic review settings February 27, 2026 21:40
@github-actions github-actions Bot added the ci ⬛ CI/CD, build, and infrastructure issues label Feb 27, 2026

Copilot AI 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.

Pull request overview

Removes the [skip ci] marker from the automated uv.lock update commit message so that squash-merge commit bodies can’t accidentally cause GitHub Actions to skip publish workflows on releases.

Changes:

  • Update .github/workflows/update-uv-lock.yml to commit without [skip ci] in the message.

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

@mpawlow mpawlow left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@edwinjosechittilappilly

Code Review 1

  • LGTM / Approved 👍

@github-actions github-actions Bot added the lgtm label Feb 27, 2026
@edwinjosechittilappilly edwinjosechittilappilly merged commit f6a20c2 into main Feb 28, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci ⬛ CI/CD, build, and infrastructure issues lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants