-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Docker build optimizations and miscellaneous fixes #587
Conversation
733ceeb
to
58cbf8d
Compare
b031685
to
194f165
Compare
…ppear after `pip` and `npm` steps anyways.
…rectory only contains `bower` stuff and `bower` is removed in this `RUN` step.
…ectory only contains `bower` stuff and `bower` is removed in this `RUN` step.
…and is empty afterwards.
194f165
to
7e4aa2a
Compare
@jakirkham I'm having trouble tracking your PRs - this seems to just be a merge of several other PRs. Do you mean to close all the others and replace with this one? |
Initially this was a PR with most (not all) of these changes. However, I realized I did not want to force your hand into accepting changes that you did not want. So, I broke it up into smaller PRs that you could take or leave and left this one as a combined form that you could take instead. As most of the PRs are already merged, maybe it doesn't make sense to keep this any more. Especially, if it is the source of confusion. |
Includes some optimizations for the build process.
InsecurePlatformWarning
s ( Resolve someInsecurePlatformWarning
when installing withpip
#588 ).Checks out locally. Able to build image and do some basic stuff in the notebook.