-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Upgrade to Node.js 16.20.1 #24
Conversation
8ee3ae2
to
286f2e8
Compare
Sounds good to me. If any downstream consumers break as a result, we should catch those errors in their CI, so I feel it's safe to merge and begin deploying to other Pelias repos. |
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
This is just for testing to ensure everything works before we merge pelias/docker-baseimage#24
Okay, all our repositories with Docker images (services, importers, schema, etc) now build and pass tests both with their current configuration and with Node.js 16, so it's time to merge this. We can then merge PRs to update dependencies, etc to support higher Node.js versions. We can also drop Node.js 12 from all our CI matricies. |
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
This is an empty commit that serves only to tell our CI to build new Docker images now that we have merged pelias/docker-baseimage#24 and move the default Node.js version from 12 to 16 Connects pelias/pelias#950
Our Docker baseimage is currently on Node.js 12, the lowest version we support. It just entered LTS, so there's no real need to rush on removing it, but Node.js 16 has some nice new features and seemingly better performance.
We've been testing all our repositories on Node.js 16 so things should be pretty smooth. Before merging this I'll perform some additional testing.