Skip to content

Publish releases to Open VSX via Trusted Publishing in release.yml - #18

Merged
DanTup merged 2 commits into
Dart-Code:masterfrom
DanTupBot:issue6375_trusted_publishing
Nov 9, 2026
Merged

DanTup merged 2 commits into
Dart-Code:masterfrom
DanTupBot:issue6375_trusted_publishing

Conversation

@DanTupBot

Copy link
Copy Markdown
Collaborator

Implements Dart-Code/Dart-Code#6375 for Dart-Code/Flutter (matching the changes in Dart-Code/Dart-Code):

  • Adds ovsx to devDependencies.
  • Adds a Publish to Open VSX step to .github/workflows/release.yml using OIDC Trusted Publishing (--trusted-publishing) and --skip-duplicate to allow safe retries.
  • Prints the ovsx publish command (after verifying OIDC environment variables) when dry_run is enabled.
  • Updates Print retry instructions on failure to trigger if either publish step (vsce or ovsx) succeeded before a later step failed, and aligns comments in release.yml with Dart-Code/Dart-Code.

@DanTupBot
DanTupBot requested a review from DanTup November 9, 2026 12:18

@DanTupBot DanTupBot left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM!

  • ovsx (^1.4.0) is added to devDependencies (compatible with Node 22 used in the workflow).
  • The Publish to Open VSX step matches the VS Code Marketplace step structure (including OIDC environment verification during dry_run, --trusted-publishing, and --skip-duplicate for safe retries, while omitting --pre-release so ovsx reads the flag from the .vsix manifest).
  • Failure retry instructions properly check both publish_vscode and publish_openvsx outcomes, and .github/workflows/release.yml is kept in sync with Dart-Code/Dart-Code.

Comment thread .github/workflows/release.yml Resolved
@DanTup
DanTup added this pull request to the merge queue Nov 9, 2026
Merged via the queue into Dart-Code:master with commit b9d60b2 Nov 9, 2026
4 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