Skip to content

Dev gbrueckl#217

Merged
gbrueckl merged 5 commits intomainfrom
dev_gbrueckl
Aug 29, 2025
Merged

Dev gbrueckl#217
gbrueckl merged 5 commits intomainfrom
dev_gbrueckl

Conversation

@gbrueckl
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings August 29, 2025 11:44
@gbrueckl gbrueckl merged commit 96e2d9f into main Aug 29, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the VSCode extension to version 2.8.2 and adds automated deployment support for OpenVSX registry alongside the existing VSCode Marketplace deployment.

  • Bumps version from 2.8.1 to 2.8.2
  • Adds OpenVSX automated deployment to the GitHub workflow
  • Reorganizes the workflow to publish to VSCode Marketplace first, then OpenVSX

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
package.json Updates version number from 2.8.1 to 2.8.2
CHANGELOG.md Adds release notes for v2.8.2 documenting OpenVSX deployment addition
.github/workflows/create_release_and_publish.yaml Adds OpenVSX publishing step, debug output, and reorders publishing workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +44 to +45
pat: ${{ secrets.OPENVSX_PUBLISH_TOKEN }}

Copy link

Copilot AI Aug 29, 2025

Choose a reason for hiding this comment

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

[nitpick] The condition logic is inconsistent with the removed step at the bottom of the file. The original condition used ${{ inputs.publish == 'true' }} || ${{ github.event_name == 'push' }} with separate expression blocks, but this uses a single expression block. Consider maintaining consistency in conditional formatting.

Suggested change
pat: ${{ secrets.OPENVSX_PUBLISH_TOKEN }}
if: ${{ inputs.publish == 'true' }} || ${{ github.event_name == 'push' }}

Copilot uses AI. Check for mistakes.
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