-
Notifications
You must be signed in to change notification settings - Fork 295
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
chore(release): publish v2.0.0-alpha-prerelease #2422
Conversation
731d555
to
2f86c9f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sandeepnRES LGTM - there'll most likely be issues with this one but we are on the right track to hammer out the details. Thank you for all the work you are putting in!
836b848
to
abde865
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
8db6f4b
to
cc04bb9
Compare
cc04bb9
to
e5a5c91
Compare
e5a5c91
to
e5664da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thankyou for the PR !
62b123d
to
5256f75
Compare
chore: added weaver packages in lerna.json feat(weaver): added script to update weaver versions fix: fix go-gen-checksum to work with v2 or above versions fix(ci): go pre release test fix chore: make all package licenses Apache-2.0 Signed-off-by: Sandeep Nishad <[email protected]> Signed-off-by: Peter Somogyvari <[email protected]> Co-authored-by: Peter Somogyvari <[email protected]>
3521c89
to
a7da881
Compare
Testing release with
alpha-prerelease
.Have to change
gomodule
names due to golang mandatory requirement that any version with v2 or above, must have major version suffixed to thegomodule
name. Also had to modifygo-gen-checksum
script according to this convention.