[openstreetmap/openstreetmap-website] Make Docker Image use Node.js v18 (PR #4992)
Emin Kocan
notifications at github.com
Mon Jul 15 14:56:57 UTC 2024
This PR addresses #4989.
Errors are encountered when attempting to run `bundle exec rails eslint` inside the Docker container terminal,after following the instructions in the `DOCKER.md` file.
After updating `DOCKER.md` to use Node.js v18, we can rebuild the container using `docker compose up --build -d ` command. Resulting in expected result when running `bundle exec rails eslint`
```bash
$ bundle exec rails eslint
warning: parser/current is loading parser/ruby30, which recognizes 3.0.7-compliant syntax, but you are running 3.0.2.
Please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
yarn run v1.22.22
$ /app/node_modules/.bin/eslint -c /app/config/eslint.js /app/app/assets/config/manifest.js /app/app/assets/javascripts/application.js /app/app/assets/javascripts/auth_providers.js /app/app/assets/javascripts/diary_entry.js /app/app/assets/javascripts/fixthemap.js /app/app/assets/javascripts/id.js /app/app/assets/javascripts/index/changeset.js /app/app/assets/javascripts/index/contextmenu.js /app/app/assets/javascripts/index/directions/fossgis_osrm.js /app/app/assets/javascripts/index/directions/fossgis_valhalla.js /app/app/assets/javascripts/index/directions/graphhopper.js /app/app/assets/javascripts/index/directions.js /app/app/assets/javascripts/index/export.js /app/app/assets/javascripts/index/history.js /app/app/assets/javascripts/index/layers/data.js /app/app/assets/javascripts/index/layers/notes.js /app/app/assets/javascripts/index/new_note.js /app/app/assets/javascripts/index/note.js /app/app/assets/javascripts/index/query.js /app/app/assets/javascripts/index/search.js /app/app/assets/javascripts/index.js /app/app/assets/javascripts/leaflet.key.js /app/app/assets/javascripts/leaflet.layers.js /app/app/assets/javascripts/leaflet.locate.js /app/app/assets/javascripts/leaflet.map.js /app/app/assets/javascripts/leaflet.note.js /app/app/assets/javascripts/leaflet.query.js /app/app/assets/javascripts/leaflet.share.js /app/app/assets/javascripts/leaflet.sidebar-pane.js /app/app/assets/javascripts/leaflet.sidebar.js /app/app/assets/javascripts/leaflet.zoom.js /app/app/assets/javascripts/login.js /app/app/assets/javascripts/matomo.js /app/app/assets/javascripts/messages.js /app/app/assets/javascripts/oauth.js /app/app/assets/javascripts/richtext.js /app/app/assets/javascripts/router.js /app/app/assets/javascripts/user.js /app/app/assets/javascripts/welcome.js /app/config/eslint.js
/app/app/assets/javascripts/index.js
267:9 warning Unexpected alert no-alert
/app/app/assets/javascripts/index/directions.js
117:11 warning Unexpected alert no-alert
337:9 warning Unexpected 'todo' comment: 'TODO: collapse width of sidebar back to...' no-warning-comments
/app/app/assets/javascripts/index/directions/fossgis_osrm.js
4:1 warning Unexpected function declaration in the global scope, wrap in an IIFE for a local variable, assign as global property for a global variable no-implicit-globals
/app/app/assets/javascripts/index/directions/fossgis_valhalla.js
1:1 warning Unexpected function declaration in the global scope, wrap in an IIFE for a local variable, assign as global property for a global variable no-implicit-globals
/app/app/assets/javascripts/index/directions/graphhopper.js
1:1 warning Unexpected function declaration in the global scope, wrap in an IIFE for a local variable, assign as global property for a global variable no-implicit-globals
68:17 warning Unexpected 'todo' comment: 'TODO does graphhopper map instructions...' no-warning-comments
✖ 7 problems (0 errors, 7 warnings)
Done in 0.99s.
#
```
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/4992
-- Commit Summary --
* make docker image use nodejs v18
-- File Changes --
M Dockerfile (12)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/4992.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4992.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4992
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4992 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240715/405b268c/attachment-0001.htm>
More information about the rails-dev
mailing list