Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use the extension version in download URL for binary. #999

Merged
merged 1 commit into from
May 28, 2024

Conversation

rgrunber
Copy link
Member

  • The language server version was used as the version for the download URL of the binary, but since the binaries are hosted under the vscode-xml release tab, it should match the vscode-xml release
  • Remove some unnecessary steps from the release job

I also slightly refactored the ordering. The preparing of the pre-release affects the final version number, so extracting the version number must happen after. Any workflow that users the versions can occur immediately after.

Also set xmlServer/version back to 0.28.0 as we may end up doing a bug fix release for this, using the same language server.

- The language server version was used as the version for the download
  URL of the binary, but since the binaries are hosted under the
  vscode-xml release tab, it should match the vscode-xml release
- Remove some unnecessary steps from the release job

Signed-off-by: Roland Grunberg <[email protected]>
@rgrunber rgrunber requested a review from datho7561 May 28, 2024 18:13
Copy link
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Seems good to me

@datho7561 datho7561 merged commit 3884fa5 into redhat-developer:main May 28, 2024
1 check passed
@rgrunber rgrunber deleted the fix-gha-publish branch May 28, 2024 20:05
@rgrunber rgrunber added this to the 0.27.1 milestone May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants