andygrove commented on code in PR #403: URL: https://github.com/apache/arrow-ballista/pull/403#discussion_r1001856658
########## dev/docker/ballista-scheduler.Dockerfile: ########## @@ -24,7 +24,16 @@ ENV RUST_LOG=info ENV RUST_BACKTRACE=full ENV DEBIAN_FRONTEND=noninteractive -RUN apt-get update && apt-get install -y nginx netcat +RUN apt-get update &&\ + apt-get install -y nginx netcat nodejs npm + +COPY ballista/ui /root/ballista/ui + +RUN cd /root/ballista/ui/scheduler &&\ Review Comment: > The CI scripts are the canonical place to look to see how to run a build, and they are under test. For expert developers such as yourself, I can see how this makes sense, but I want to make sure that we are inclusive for all levels and backgrounds. Some developers would feel intimidated if they were expected to learn how CI works just to be able to build the project locally and would prefer to have documentation and scripts, especially with a project this complex, with multiple languages & technologies (Python, Rust, React, Docker). I know I have given up on projects if I couldn't figure out how to build them locally without going through new learning curves. I'll file an issue for Windows support. It would be great to support that. -- 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]
