Skip to content

Fix master client Docker build: Vue 3.5 + yarn Dockerfile - #279

Merged
ywkim312 merged 2 commits into
masterfrom
fix/278-fix-master-client-docker-build-vue-vue-router-peer-dependency-conflict
Jul 27, 2026
Merged

Fix master client Docker build: Vue 3.5 + yarn Dockerfile#279
ywkim312 merged 2 commits into
masterfrom
fix/278-fix-master-client-docker-build-vue-vue-router-peer-dependency-conflict

Conversation

@ywkim312

Copy link
Copy Markdown
Contributor

Summary

  • Fix failing Dockerize Vue Client on master caused by npm install resolving vue-router@4.6.x against vue@~3.3.4
  • Bump vue, @vue/compat, and @vue/compiler-sfc to ~3.5.22
  • Switch client/Dockerfile to yarn + yarn.lock (stop using npm install in the image build)
  • Update yarn.lock / PnP files accordingly

Why

This was exposed when #219 rebuilt master; that PR did not change deps. master already had an outdated client Docker/npm setup. dev already uses Vue 3.5 + yarn.

Test plan

  • yarn install succeeds locally
  • yarn build succeeds locally
  • Dockerize Vue Client passes on this branch / after merge to master
  • Client image tags for master / latest publish successfully

Notes

  • Target branch: master
  • No need to merge this into dev as a primary fix (dev already has Vue 3.5 + yarn Dockerfile)

@ywkim312 ywkim312 self-assigned this Jul 17, 2026
@ywkim312 ywkim312 linked an issue Jul 17, 2026 that may be closed by this pull request
7 tasks
@ywkim312
ywkim312 requested review from lmarini and valentinedwv July 17, 2026 17:49

@valentinedwv valentinedwv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are the client/.pnp.* files needed in the repo?
Yes they should be

@ywkim312
ywkim312 merged commit 6da8d57 into master Jul 27, 2026
3 checks passed
@ywkim312
ywkim312 deleted the fix/278-fix-master-client-docker-build-vue-vue-router-peer-dependency-conflict branch July 27, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix master client Docker build: Vue / vue-router peer dependency conflict

2 participants