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

Build pre-release VSIX in CI #1450

Merged
merged 5 commits into from
Mar 20, 2025

Conversation

matthewbastien
Copy link
Member

@matthewbastien matthewbastien commented Mar 20, 2025

Update the preview-package script to generate a pre-release package with one minor version greater than the current version from the package.json and a patch number set to today's date formatted as YYYYMMddhhmm. The nightly CI will now generate both a release and pre-release extension VSIX. This enables us to create a pre-release version of the Swift extension.

Going forward, we'll have to abide by the following rules in order for this to work:

  1. The version in the package.json should always be the current release version of the extension as published on the VS Code marketplace.
  2. Always use even numbered minor versions for releases. Pre-release builds will then always have an odd minor version.

I've added a check in CI to enforce even numbered minor versions in the package.json.

@matthewbastien matthewbastien added the full-test-run Perform a full test suite run. Apply this label before pushing up a PR or commit label Mar 20, 2025
Copy link
Contributor

@rbenegal rbenegal left a comment

Choose a reason for hiding this comment

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

Looks great to me!

@matthewbastien matthewbastien merged commit 487d943 into swiftlang:main Mar 20, 2025
13 of 15 checks passed
@matthewbastien matthewbastien deleted the pre-release-script branch March 20, 2025 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-test-run Perform a full test suite run. Apply this label before pushing up a PR or commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants