8000 DOC: update CI environment dependencies for doc build by Dawnfz-Lenfeng · Pull Request #814 · xorbitsai/xorbits · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

DOC: update CI environment dependencies for doc build #814

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 25 commits into from
Sep 19, 2024
Merged

DOC: update CI environment dependencies for doc build #814

merged 25 commits into from
Sep 19, 2024

Conversation

Dawnfz-Lenfeng
Copy link
Contributor

What do these changes do?

  • Updated the doc build environment dependencies, separating them from other module dependencies.
  • Handled the absence of lightgbm by setting import lightgbm to None, preventing import errors when it's not installed.

Related issue number

Fixes #xxxx

Check code requirements

  • tests added / passed (if needed)
  • Ensure all linting tests pass

@XprobeBot XprobeBot added the documentation Improvements or additions to documentation label Sep 19, 2024
@XprobeBot XprobeBot added this to the v0.7.4 milestone Sep 19, 2024
@codecov Codecov
Copy link
codecov bot commented Sep 19, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.97%. Comparing base (c765e57) to head (aecfb9f).
Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #814   +/-   ##
=======================================
  Coverage   82.97%   82.97%           
=======================================
  Files        1070     1070           
  Lines       80031    80033    +2     
  Branches    16528    16529    +1     
=======================================
+ Hits        66406    66410    +4     
+ Misses      12030    12023    -7     
- Partials     1595     1600    +5
Flag Coverage Δ
unittests 82.88% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@luweizheng luweizheng merged commit dd46556 into xorbitsai:main Sep 19, 2024
31 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luweizheng luweizheng luweizheng approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Dawnfz-Lenfeng @luweizheng @XprobeBot
0