-
Notifications
You must be signed in to change notification settings - Fork 884
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
Update documentation requirements to latest version #2225
Conversation
gsheni
commented
Aug 5, 2022
•
edited
Loading
edited
- You can preview the docs here:
- https://readthedocs.com/projects/feature-labs-inc-featuretools/builds/1044856/
Codecov Report
@@ Coverage Diff @@
## main #2225 +/- ##
==========================================
- Coverage 99.29% 99.29% -0.01%
==========================================
Files 146 146
Lines 17583 17577 -6
==========================================
- Hits 17459 17453 -6
Misses 124 124
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
I was originally going to use a .readthedocs.yml file as below. However, this resulted in a significant increase in ru ntime 455 seconds to 594 seconds. Therefore I kept the existing format. It seems strange that readthedocs has the Spark requirements, graphviz, and pandoc installed (without specifying it in .readthedocs.yaml) # .readthedocs.yaml
# Read the Docs configuration file
# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
# Required
version: 2
# Set the version of Python and other tools you might need
build:
os: ubuntu-20.04
tools:
python: "3.8"
apt_packages:
- pandoc
- graphviz
- openjdk-11-jre-headless
jobs:
post_create_environment:
- export PYARROW_IGNORE_TIMEZONE=1
- export JAVA_HOME="/usr/lib/jvm/java-11-openjdk-amd64"
# Build documentation in the docs/ directory with Sphinx
sphinx:
configuration: docs/source/conf.py
# Optionally build your docs in additional formats such as PDF and ePub
formats: []
# Optionally set the version of Python and requirements required to build your docs
python:
install:
- method: pip
path: .
extra_requirements:
- docs |
|
Co-authored-by: Shripad Badithe <60528327+sbadithe@users.noreply.github.com>
Co-authored-by: Shripad Badithe <60528327+sbadithe@users.noreply.github.com>
Co-authored-by: Shripad Badithe <60528327+sbadithe@users.noreply.github.com>
@@ -13,6 +13,7 @@ Future Release | |||
* Documentation Changes | |||
* Testing Changes | |||
* Update create feedstock PR forked repo to use (:pr:`2223`) | |||
* Update development requirements and use latest for documentation (:pr:`2225`) | |||
|
|||
Thanks to the following people for contributing to this release: | |||
:user:`gsheni`, :user:`ozzieD`, user:`sbadithe` |
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.
:user:`gsheni`, :user:`ozzieD`, user:`sbadithe` | |
:user:`gsheni`, :user:`ozzieD`, :user:`sbadithe` |
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, left one suggested change