Skip to content

fix(ci): scope vscode publish workflow to extension tags#16

Merged
eduardbar merged 1 commit into
masterfrom
fix/vscode-release-trigger-scope
Mar 18, 2026
Merged

fix(ci): scope vscode publish workflow to extension tags#16
eduardbar merged 1 commit into
masterfrom
fix/vscode-release-trigger-scope

Conversation

@eduardbar

Copy link
Copy Markdown
Owner

Summary

  • remove ineffective release tag filter under release trigger
  • gate the VS Code publish job to run only for vscode-v* tags or manual dispatch
  • keep version verification active for both release and workflow_dispatch paths

Why

Core drift releases use vX.Y.Z tags and should not fail the VS Code extension workflow, which expects vscode-vX.Y.Z tags.

Limit the VS Code publish job to vscode-v* tag releases (or manual dispatch) so core drift releases do not fail due to extension version mismatch.
@github-actions

Copy link
Copy Markdown

drift trust

drift trust

  • Trust Score: 78/100
  • Merge Risk: MEDIUM
  • Target: /home/runner/work/drift/drift

Diff signals

  • Base ref: origin/master
  • Diff status: NEUTRAL
  • Score delta: +0
  • Issues: +0 new / -0 resolved
  • Trust adjustment: +0 penalty / -0 bonus (net +0)

Top reasons

  • Maintenance hotspots: Maintenance risk is MEDIUM (35/100). (impact 9)
  • Error-level issues: 2 error issue(s) increase merge volatility. (impact 8)
  • Highest-risk file: /home/runner/work/drift/drift/src/guard.ts has hotspot risk 35/100. (impact 5)

Fix priorities

drift review

drift review

ℹ️ No drift changes detected against base ref.

  • Base ref: origin/master
  • Score delta: +0
  • New issues: 0
  • Resolved issues: 0

File breakdown

  • No file-level deltas detected

@eduardbar eduardbar merged commit fd33031 into master Mar 18, 2026
2 checks passed
@eduardbar eduardbar deleted the fix/vscode-release-trigger-scope branch March 18, 2026 20:41
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.

1 participant