-
Notifications
You must be signed in to change notification settings - Fork 60
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
crud: bump rocks minor version with compatibility fixes #339
crud: bump rocks minor version with compatibility fixes #339
Conversation
.github/workflows/testing.yml
Outdated
if: matrix.tarantool == 'pr/8630' | ||
run: | | ||
tt init | ||
tt install tarantool pr/8630 |
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.
2.x latest setup uses sudo tt install
, you may need this too.
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.
The workflow succeeded, looks like sudo
is unnecessary.
CHANGELOG.md
Outdated
@@ -20,6 +20,7 @@ Versioning](http://semver.org/spec/v2.0.0.html) except to the first release. | |||
- Support `fetch_latest_metadata` option for crud requests with metadata (#335) | |||
- Support `noreturn` option for data change crud requests (#335) | |||
- Support `crud.schema` request (#336) | |||
- Support Tarantool tuples over IPROTO in crud (#337) |
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.
go-tarantool does not depends on crud, the fix is related only to the test infrastructure
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.
Fixed.
55967a6
to
47e255e
Compare
47e255e
to
68269dd
Compare
Thank you for the patch! |
Needed for tarantool/tarantool#8147
Related issues: