Skip to content
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: add e2e tests for new SDK #4352

Merged
merged 9 commits into from
Dec 25, 2023
Merged

feat: add e2e tests for new SDK #4352

merged 9 commits into from
Dec 25, 2023

Conversation

frostming
Copy link
Contributor

Signed-off-by: Frost Ming [email protected]

What does this PR address?

Fixes #(issue)

Before submitting:

@frostming frostming requested a review from a team as a code owner December 21, 2023 10:22
@frostming frostming requested review from parano and removed request for a team December 21, 2023 10:22
@frostming frostming force-pushed the e2e-test branch 3 times, most recently from 350211d to bd690d7 Compare December 22, 2023 04:57
Signed-off-by: Frost Ming <[email protected]>
Signed-off-by: Frost Ming <[email protected]>
Signed-off-by: Frost Ming <[email protected]>
Signed-off-by: Frost Ming <[email protected]>
Signed-off-by: Frost Ming <[email protected]>
Signed-off-by: Frost Ming <[email protected]>
Copy link

Code Coverage

Package Statements
bentoml 64.29% (10571 / 16441)
Summary 64.29% (10571 / 16441)

@frostming
Copy link
Contributor Author

All CI went green 🎉 , except for the annoying FOSSA 😢 . Is it possible to add me as the FOSSA maintainer to see if we can fix them? Also there are some flaky tests regarding framework model inference.

/cc @ssheng @aarnphm

@frostming frostming merged commit d1e44c0 into 1.2 Dec 25, 2023
38 checks passed
@frostming frostming deleted the e2e-test branch December 25, 2023 02:54
frostming added a commit that referenced this pull request Jan 17, 2024
* feat: add e2e tests for new SDK

Signed-off-by: Frost Ming <[email protected]>

* fix:test setup

Signed-off-by: Frost Ming <[email protected]>

* feat: add new sdk tests

Signed-off-by: Frost Ming <[email protected]>

* fix: typing compatibility

Signed-off-by: Frost Ming <[email protected]>

* fix: compatibility

Signed-off-by: Frost Ming <[email protected]>

* fix: adjust test data

Signed-off-by: Frost Ming <[email protected]>

* fix: change the verify data

Signed-off-by: Frost Ming <[email protected]>

* fixup

Signed-off-by: Frost Ming <[email protected]>

* fix: function signature and defaults

Signed-off-by: Frost Ming <[email protected]>
frostming added a commit that referenced this pull request Feb 1, 2024
* feat: add e2e tests for new SDK

Signed-off-by: Frost Ming <[email protected]>

* fix:test setup

Signed-off-by: Frost Ming <[email protected]>

* feat: add new sdk tests

Signed-off-by: Frost Ming <[email protected]>

* fix: typing compatibility

Signed-off-by: Frost Ming <[email protected]>

* fix: compatibility

Signed-off-by: Frost Ming <[email protected]>

* fix: adjust test data

Signed-off-by: Frost Ming <[email protected]>

* fix: change the verify data

Signed-off-by: Frost Ming <[email protected]>

* fixup

Signed-off-by: Frost Ming <[email protected]>

* fix: function signature and defaults

Signed-off-by: Frost Ming <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant