nchammas opened a new pull request #27534: [SPARK-30731][DOCS] Refine workflow 
for building docs
URL: https://github.com/apache/spark/pull/27534
 
 
   ### What changes were proposed in this pull request?
   
   This PR makes the following refinements to the workflow for building docs:
   * Install Python and Ruby consistently using pyenv and rbenv across both the 
docs README and the release Dockerfile.
   * Pin the Python and Ruby versions we use.
   * Pin all direct Python and Ruby dependency versions.
   * Eliminate any use of `sudo pip` or `sudo gem`, which the Python community 
discourages.
   * Remove unused cruft in the Python docs Makefiles.
   * Update a deprecated mkdocs config.
   
   ### Why are the changes needed?
   
   This PR should increase the consistency and reproducibility of the 
doc-building process by managing Python and Ruby in a more consistent way, and 
by eliminating unused or outdated code.
   
   Here's a possible example of an issue building the docs that would be 
addressed by the changes in this PR: 
https://github.com/apache/spark/pull/27459#discussion_r376135719
   
   ### Does this PR introduce any user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Manual tests:
   * I was able to build the Docker image successfully, minus the final part 
about `RUN useradd`.
       * I am unable to run `do-release-docker.sh` because I am not a committer 
and don't have the required GPG key.
   * I built the docs locally and viewed them in the browser.
   
   I think I need a committer to more fully test out these changes.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to