-
Notifications
You must be signed in to change notification settings - Fork 2
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
feature(release): add CI for releasing and testing #32
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Dusan Malusev <[email protected]>
5cee2ab
to
016effe
Compare
@CodeLieutenant left to fix the tag matching, everything else LGTM |
What should I do with -scylladb suffix in tags. docker images will have that in the version. Should it be stripped out? |
The Keep it for now. |
I kept it. Also updated all github actions to use maintained setup-rust toolchain action |
fada357
to
10c53e5
Compare
Signed-off-by: Dusan Malusev <[email protected]>
10c53e5
to
e410bea
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
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.
Please, squash changes into one commit (move second commit changes to the first one.)
gemini