elgalu / docker-selenium

[NOT MAINTAINED] Please use <https://github.com/SeleniumHQ/docker-selenium>
https://github.com/SeleniumHQ/docker-selenium
Other
1.42k stars 334 forks source link

Generic cleanup #347

Closed TobiX closed 4 years ago

TobiX commented 4 years ago

These are some suggestions, mainly to get some discussion going :smile_cat:

I think having a more "maintainable" Dockerfile would lower the barrier for external contributors, so here are some minor "clean-up" suggestions:

I don't expect this to be merged without comment, so: Roast me!

PS: I would even suggest to go quite a bit further with the clean-up: Currently, there are 13 steps which use apt to install packages. I would suggest joining them in one big step (layer) and move all the configuration after that. That would also make it easier to unify the use of --no-install-recommends, which is currently missing in some places, which probably also bloats the image a bit.

elgalu commented 4 years ago

Nice! thanks! if the build passes I think we can merge<3

elgalu commented 4 years ago

:+1:

TobiX commented 4 years ago

if the build passes I think we can merge

Sure, if you are fine with all my changes :+1:

elgalu commented 4 years ago

let's wait a bit to see if @diemol remembers something about those issues else I think we can merge.

diemol commented 4 years ago

Honestly, all the changes affect code that was there even before I started helping with the repo, probably all added by @elgalu, so if he is fine with it, I am fine as well.

I would only ask, if @TobiX, could stick around after the following releases, in case something does not work and we need some help to debug it and fix it.

TobiX commented 4 years ago

I would only ask, if @TobiX, could stick around after the following releases, in case something does not work and we need some help to debug it and fix it.

We're actively using this image, so I should be able to notice any problems 😅