-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Scylla new integration #5440
Scylla new integration #5440
Conversation
Codecov Report
|
acae873
to
f43cf68
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.
Great work!!
A couple of comments/suggestions, mostly nits. Exciting stuff!
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.
💯
78a1f03
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.
Excellent!
Early commit for Python 3.8, those checks are expected to be failing right now. |
What does this PR do?
Add new integration Scylla.
Motivation
To expand integration coverage.
Additional Notes
TODO (in separate PRs):
Metadata collectionDoneReview checklist (to be filled by reviewers)
changelog/
andintegration/
labels attached