-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Skip tests that depend on OpenAI via --skip-openai
#1097
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1097 +/- ##
==========================================
+ Coverage 30.56% 40.49% +9.92%
==========================================
Files 30 30
Lines 4028 4033 +5
Branches 911 962 +51
==========================================
+ Hits 1231 1633 +402
+ Misses 2723 2289 -434
- Partials 74 111 +37
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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! Thanks!
* add doc for spark * labelCol equals to label by default * change title and reformat * reference about default index type * fix doc build * Update website/docs/Examples/Integrate - Spark.md * update doc * Added more references * remove exception case when `y_train.name` is None * fix broken link --------- Co-authored-by: Wendong Li <[email protected]> Co-authored-by: Li Jiang <[email protected]>
* --skip-openai * All tests pass * Update build.yml * Update Contribute.md * Fix for failing Ubuntu tests * More tests skipped, fixing 3.10 build * Apply suggestions from code review Co-authored-by: Qingyun Wu <[email protected]> * Added more comments * fixed test__wrap_function_* --------- Co-authored-by: Qingyun Wu <[email protected]> Co-authored-by: Davor Runje <[email protected]>
Allows skipping test that depend on OpenAI. It is possible to run the whole test suite with
openai
dependency installed viapytest test --skip-openai
(previously it was required to uninstallopenai
lib to run tests that don't interact with LLM).Related issue number
Partially closes #1088
Checks