paolorechia commented on PR #718: URL: https://github.com/apache/arrow-ballista/pull/718#issuecomment-1492430696
Hi, @jnaous I don't have a strong preference here, I just wanted to fix the build so I could move forward with docker build workflow: https://github.com/apache/arrow-ballista/issues/162 One advantage I see with your PR is that you use Node 18 as opposed to Node 16 from my PR, as I ran into issues with the OpenSSL library. It seems that you solved this issue with Node 18 it by adding `--openssl-legacy-provider` into the react build command, so it should also be easy to reuse your solution with my NVM PR. Either way, both approaches work for me. -- 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]
