nchammas commented on PR #57010:
URL: https://github.com/apache/spark/pull/57010#issuecomment-5530947891

   The `build-docs` script added in #48860 does not run as part of CI and is 
[currently 
broken](https://github.com/apache/spark/pull/58352#issuecomment-5455088866). 
This PR here refactors a skip flag added by that same PR.
   
   I personally think it's best if we just remove `build-docs` or replace it 
with something that runs regularly on CI so we are not repeatedly discovering 
that it broke. But I don't know if others would support that. Because I am not 
sure about the future of `build-docs`, I will abandon this PR for now.
   
   I will reintroduce `print_skip_flags` in a separate PR since that's useful 
by itself.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to