Skip to content

bump up version to v0.10.0-alpha.3#301

Merged
SteveLasker merged 2 commits intomainfrom
yizha1/prepare-release
Aug 17, 2022
Merged

bump up version to v0.10.0-alpha.3#301
SteveLasker merged 2 commits intomainfrom
yizha1/prepare-release

Conversation

@yizha1
Copy link
Contributor

@yizha1 yizha1 commented Aug 12, 2022

Signed-off-by: Yi Zha zhayi@outlook.com

@codecov-commenter
Copy link

codecov-commenter commented Aug 12, 2022

Codecov Report

Merging #301 (252d78c) into main (3ced0c9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #301   +/-   ##
=======================================
  Coverage   30.64%   30.64%           
=======================================
  Files          25       25           
  Lines        1612     1612           
=======================================
  Hits          494      494           
  Misses       1105     1105           
  Partials       13       13           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: Yi Zha <zhayi@outlook.com>
@yizha1 yizha1 force-pushed the yizha1/prepare-release branch from 7d6ebff to 95fad0d Compare August 12, 2022 05:16
Copy link
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

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

LGTM

@yizha1 yizha1 requested a review from a team August 12, 2022 05:18
Copy link
Contributor

@gokarnm gokarnm left a comment

Choose a reason for hiding this comment

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

LGTM!

var (
// Version shows the current notation version, optionally with pre-release.
Version = "v0.7.1-alpha.1"
Version = "v1.0.0-alpha.3"
Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably not bump major version to v1.0.0 right now, the alpha releases will have patch updates. Which will bump 1.0.0-{prereleasetag} to 1.0.x-{prereleasetag} and require the v1 release to use the same patch version at some point, this would be bit confusing (e.e. v1.0.5-alpha.3 vs 1.0.5)

https://stackoverflow.com/questions/51373151/releasing-alpha-to-beta-to-production-with-semver

Suggested change
Version = "v1.0.0-alpha.3"
Version = "v0.8.0-alpha.3"

Copy link
Contributor

@gokarnm gokarnm left a comment

Choose a reason for hiding this comment

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

Suggesting bumping to v0.8.0-alpha.3 instead.

@yizha1
Copy link
Contributor Author

yizha1 commented Aug 15, 2022

Suggesting bumping to v0.8.0-alpha.3 instead.

@gokarnm I think it should be v0.9.0-alpha.3. We already released v0.9.0-alpha.1 if you check the release page. The version file was not updated by previous release.

@priteshbandi
Copy link
Contributor

Can we do v0.9.0-alpha.3 ?

Signed-off-by: Yi Zha <zhayi@outlook.com>
@yizha1 yizha1 changed the title bump up version to v1.0.0-alpha.3 bump up version to v0.10.0-alpha.3 Aug 16, 2022
@yizha1
Copy link
Contributor Author

yizha1 commented Aug 16, 2022

Can we do v0.9.0-alpha.3 ?

v0.10.0-alpha.3 is better, since there are features merged, so the minor version should be stepped.

Copy link
Contributor

@priteshbandi priteshbandi left a comment

Choose a reason for hiding this comment

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

LGTM.

@dtzar
Copy link
Contributor

dtzar commented Aug 16, 2022

@gokarnm - it is blocked due to your comment not being resolved. You have a LGTM, but no approval.

Copy link
Contributor

@gokarnm gokarnm left a comment

Choose a reason for hiding this comment

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

LGTM!

@SteveLasker SteveLasker merged commit 87aa9c5 into main Aug 17, 2022
@dtzar dtzar deleted the yizha1/prepare-release branch August 17, 2022 22:37
chloeyin added a commit that referenced this pull request Aug 25, 2022
* doc: add missing username/password options to commands (#293)

Signed-off-by: Binbin Li <libinbin@microsoft.com>

* bump up version to v0.10.0-alpha.3 (#301)

* bump up version to v1.0.0-alpha.3
* revise version to v0.10.0-alpha.3

Signed-off-by: Yi Zha <zhayi@outlook.com>

* fix: update notation-go (#294)

Signed-off-by: Junjie Gao <junjiegao@microsoft.com>

* Build: Bump dependencies (#306)

Signed-off-by: Yi Zha <zhayi@outlook.com>

* feat: add weekly release (#282)

Signed-off-by: Junjie Gao <junjiegao@microsoft.com>

Signed-off-by: Binbin Li <libinbin@microsoft.com>
Signed-off-by: Yi Zha <zhayi@outlook.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Co-authored-by: Binbin Li <libinbin050215@gmail.com>
Co-authored-by: Yi Zha <107919912+yizha1@users.noreply.github.com>
Co-authored-by: Junjie Gao <43160897+JeyJeyGao@users.noreply.github.com>
Co-authored-by: zaihaoyin <zaihaoyin@microsoft.com>
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
* bump up version to v1.0.0-alpha.3
* revise version to v0.10.0-alpha.3

Signed-off-by: Yi Zha <zhayi@outlook.com>
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.

7 participants

Comments