-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
feat: foundations #209
feat: foundations #209
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feature/website #209 +/- ##
====================================================
- Coverage 46.72% 21.87% -24.86%
====================================================
Files 486 102 -384
Lines 30818 10731 -20087
====================================================
- Hits 14399 2347 -12052
+ Misses 16419 8384 -8035
|
# "platform/api", | ||
"image-processor", | ||
"image-processor/proto", | ||
# "video/edge", | ||
# "video/ingest", | ||
# "video/transcoder", | ||
# "video/lib/*", | ||
# "video/api", | ||
# "video/player", | ||
# "video/player_types", | ||
# "video/common", | ||
# "video/cli", | ||
# "binary-helper", | ||
# "utils", | ||
# "proto", | ||
# "config", | ||
# "config/derive", |
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.
whats going on here?
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.
due to changes the other libs dont build, and we havent fixed them so in order to use i commented them out
Converts the old dataloader model into a more generic batcher for any operations
a5aefe2
to
e0eca0d
Compare
- Allow custom tls alpn - Expose socket type - Fix formatting - bump deps
Proposed changes
Describe the big picture of what you want to change here, to help the maintainers understand why we should accept this pull request. Be sure to link to the issue if it fixes a bug or resolves a feature request.
Types of changes
What types of changes does your code introduce to Scuffle?
Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Further comments
If this is a relatively large or complex change, you may want to start the discussion by explaining why you chose the solution you did, what alternatives you considered, etc.