Node version discrepancy

itzg / docker-rcon-web-admin

A Docker image that runs rcon-web-admin

MIT License

92 stars 18 forks source link

Node version discrepancy #5

Closed jabbink closed 4 years ago

jabbink commented 4 years ago

Not that I noticed any issues with the image, but I noticed you use node:10 even though the upstream requirements state to use atleast node:12.13.1 https://github.com/rcon-web-admin/rcon-web-admin#requirements

Any reason for the choice of Node 10?

itzg commented 4 years ago

I don't remember any particular reason. I'm thinking that might have been the nodejs version at the time when first integrating. I'll double check that it works fine with node:12 or node:12.18 since that's the latest of the LTS version.

itzg commented 4 years ago

Latest image is now based on 12.18 (or 12 specifically)