A subset of https://github.com/openstreetmap/openstreetmap-website/pull/6874 that might be more agreeable 😄
In several places, we call (or advise to call) Yarn with Bundler. At the very least this is two unnecessary extra words to type. More generally, this is mixing up two different runtimes, one JS and one Ruby, which work independently. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/7095 -- Commit Summary -- * No need to call Yarn within Bundler -- File Changes -- M .github/workflows/lint.yml (10) M .github/workflows/tests.yml (2) M Dockerfile (2) M doc/CONFIGURE.md (2) M doc/MANUAL_INSTALL.md (2) -- Patch Links -- https://github.com/openstreetmap/openstreetmap-website/pull/7095.patchhttps://github.com/openstreetmap/openstreetmap-website/pull/7095.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/7095 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/openstreetmap-website/pull/[email protected]>
_______________________________________________ rails-dev mailing list [email protected] https://lists.openstreetmap.org/listinfo/rails-dev
