Skip to content

Commit

Permalink
fix build scripts to properly skip voyager standalone build
Browse files Browse the repository at this point in the history
  • Loading branch information
sdumetz committed Jan 10, 2024
1 parent 1810011 commit cda6e0b
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 7 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,6 @@ jobs:
run: (cd source/voyager && npm ci --legacy-peer-deps)
- name: install ui dependencies
run: npm ci && (cd source/ui && npm ci)
- name: build voyager
run: npm run build-voyager
- name: build UI
run: npm run build-ui
build-server:
Expand Down
4 changes: 0 additions & 4 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,6 @@ COPY source/server /app/source/server
RUN npm run build-server
# outputs files in /app/source/server/dist

COPY source/voyager /app/source/voyager
RUN npm run build-voyager
# files are automatically picked up by build-ui

COPY source/ui /app/source/ui
RUN npm run build-ui
# outputs files in /app/dist
Expand Down
2 changes: 1 addition & 1 deletion source/ui/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ module.exports = function createAppConfig(env, argv={})
},
{
from: "{js,js/draco,css,language,images}/*.{js,json,wasm,css,jpg,png,svg}",
context: path.join(project, "source/voyager/dist/"),
context: path.join(project, "source/voyager/assets/"),
force: false,
priority: 1,
},
Expand Down

0 comments on commit cda6e0b

Please sign in to comment.