Skip to content

ci: make release vscode manual step#4830

Merged
benfdking merged 1 commit intomainfrom
deploy_vscode_build_manual
Jun 26, 2025
Merged

ci: make release vscode manual step#4830
benfdking merged 1 commit intomainfrom
deploy_vscode_build_manual

Conversation

@benfdking
Copy link
Copy Markdown
Contributor

No description provided.

@benfdking benfdking requested a review from Copilot June 26, 2025 16:52
Copy link
Copy Markdown
Contributor

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 pull request enables a manual release step for the VSCode extension by adding workflow_dispatch inputs and new validation steps while also replacing the automatic version extraction.

  • Added workflow_dispatch trigger with a required version input
  • Introduced a branch check and a semantic version format validation step
  • Updated steps to use the manual input version for updating package.json and publishing the extension
Comments suppressed due to low confidence (1)

.github/workflows/release_extension.yaml:25

  • [nitpick] Consider adding a comment explaining the regex for semantic version validation to help future maintainers understand its purpose and limitations.
          if ! [[ $version =~ ^[0-9]+\.[0-9]+\.[0-9]+(-[0-9A-Za-z-]+(\.[0-9A-Za-z-]+)*)?(\+[0-9A-Za-z-]+(\.[0-9A-Za-z-]+)*)?$ ]]; then

Comment thread .github/workflows/release_extension.yaml
@benfdking benfdking force-pushed the deploy_vscode_build_manual branch from 49ea64e to 0c62f7b Compare June 26, 2025 17:54
@benfdking benfdking force-pushed the deploy_vscode_build_manual branch from 0c62f7b to 7191fa6 Compare June 26, 2025 17:55
@benfdking benfdking merged commit 3d0ab45 into main Jun 26, 2025
25 checks passed
@benfdking benfdking deleted the deploy_vscode_build_manual branch June 26, 2025 19:08
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.

3 participants