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

github/workflows: use <year>.<month>.<day>.<hour> as version #12

Closed
wants to merge 1 commit into from

Conversation

hyangah
Copy link
Contributor

@hyangah hyangah commented Mar 18, 2020

Marketplace does not allow prerelease annotation.
That caused the release error like
https://github.com/golang/vscode-go/runs/517074211?check_suite_focus=true

Use the four digit semver instead. Note the current vsce package
command allows prerelease annotation but does not allow four digit
semver format currently.

And record the commit hash to the CHANGELOG.md
since we cannot rely on the version string to encode the info.

Update #5

Marketplace does not allow prerelease annotation
(but `vsce package` command allows it).
Use the four digit semver instead.

And record the commit hash to the CHANGELOG.md
since we cannot rely on the version string to encode the info.
@googlebot googlebot added the cla: yes Used by googlebot to label PRs as having a valid CLA. The text of this label should not change. label Mar 18, 2020
@gopherbot
Copy link
Collaborator

This PR (HEAD: cf71291) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/223961 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 1:

ping?


Please don’t reply on this GitHub thread. Visit golang.org/cl/223961.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

Message from Rebecca Stambler:

Patch Set 1: Code-Review+2

Ah sorry I missed this, LGTM.


Please don’t reply on this GitHub thread. Visit golang.org/cl/223961.
After addressing review feedback, remember to publish your drafts!

gopherbot pushed a commit that referenced this pull request Mar 19, 2020
Marketplace does not allow prerelease annotation.
That caused the release error like
https://github.com/golang/vscode-go/runs/517074211?check_suite_focus=true

Use the four digit semver instead. Note the current `vsce package`
command allows prerelease annotation but does not allow four digit
semver format currently.

And record the commit hash to the CHANGELOG.md
since we cannot rely on the version string to encode the info.

Update #5

Change-Id: Id45fc822bb0d5b6b6750e28bf71bbb49f3eb97be
GitHub-Last-Rev: cf71291
GitHub-Pull-Request: #12
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/223961
Reviewed-by: Rebecca Stambler <[email protected]>
@gopherbot
Copy link
Collaborator

This PR is being closed because golang.org/cl/223961 has been merged.

@gopherbot gopherbot closed this Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Used by googlebot to label PRs as having a valid CLA. The text of this label should not change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants