Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Commit

Permalink
Smaller bundle size (#117)
Browse files Browse the repository at this point in the history
* Changed Github actions to first do `npm ci`, then `npm link`

* Modified package to not bundle library dependencies into library JS

* Added: webpack config to build both umd and esm versions of the frontend library

* Changed package run to first do `npm ci`, then `npm link` also for implementation, since now it has the libraries as dependencies

* Updated package-lock

* Switch the order of `npm link` commands because the order matters
  • Loading branch information
hmuurine committed Feb 22, 2023
1 parent 49dcfff commit 6282965
Show file tree
Hide file tree
Showing 11 changed files with 609 additions and 3,898 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/create-gh-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,26 +36,26 @@ jobs:
working-directory: ./Frontend/library
run: npm run build

- name: NPM link frontend lib into ui-library
working-directory: ./Frontend/ui-library
run: npm link ../library

- name: Install ui-library deps
working-directory: ./Frontend/ui-library
run: npm ci

- name: NPM link frontend lib into ui-library
working-directory: ./Frontend/ui-library
run: npm link ../library

- name: Build frontend ui-library
working-directory: ./Frontend/ui-library
run: npm run build-all

- name: NPM link frontend and ui-library into implementations/EpicGames
working-directory: ./Frontend/implementations/EpicGames
run: npm link ../../library ../../ui-library

- name: Install implementations/EpicGames deps
working-directory: ./Frontend/implementations/EpicGames
run: npm ci

- name: NPM link frontend and ui-library into implementations/EpicGames
working-directory: ./Frontend/implementations/EpicGames
run: npm link ../../library ../../ui-library

- name: Build implementations/EpicGames
working-directory: ./Frontend/implementations/EpicGames
run: npm run build-all
Expand Down
Loading

0 comments on commit 6282965

Please sign in to comment.