Skip to content

Commit

Permalink
fix: build solara-vuetify-app with ipywidget7 and ipywidget8 support
Browse files Browse the repository at this point in the history
it seems alias in a subdependency is not working, so we have to
create two widget managers packages.
  • Loading branch information
maartenbreddels committed Jun 23, 2023
1 parent e105b00 commit 921c59d
Show file tree
Hide file tree
Showing 9 changed files with 6,466 additions and 35 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
uses: actions/cache@v3
with:
path: packages/solara-vuetify-app/dist
key: ${{ runner.os }}-js-bundle-${{ hashFiles('packages/solara-vuetify-app/src/**', 'packages/solara-vuetify-app/package.json', 'packages/solara-widget-manager/src/**', 'packages/solara-widget-manager/package.json') }}
key: ${{ runner.os }}-js-bundle-${{ hashFiles('packages/solara-vuetify-app/src/**', 'packages/solara-vuetify-app/package.json', 'packages/solara-widget-manager/src/**', 'packages/solara-widget-manager/package.json', 'packages/solara-widget-manager8/src/**', 'packages/solara-widget-manager8/package.json') }}

- name: Build solara widget manager
if: steps.cache-js-bundle.outputs.cache-hit != 'true'
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/release_solara_vuetify_app.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@ jobs:
npm install
npm run build
cd ..
cd solara-widget-manager8
npm install
npm run build
cd ..
cd solara-vuetify-app
npm install
npm run build
Expand Down
39 changes: 11 additions & 28 deletions packages/solara-vuetify-app/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 2 additions & 6 deletions packages/solara-vuetify-app/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,7 @@ module.exports = [
},
resolve: {
alias: {
"@jupyter-widgets/base": "@jupyter-widgets/base8",
"@jupyter-widgets/controls": "@jupyter-widgets/controls8",
"@jupyter-widgets/jupyterlab-manager": "@jupyter-widgets/jupyterlab-manager8",
"@widgetti/solara-widget-manager": "@widgetti/solara-widget-manager8",
}
},
mode: 'production',
Expand All @@ -93,9 +91,7 @@ module.exports = [
},
resolve: {
alias: {
"@jupyter-widgets/base": "@jupyter-widgets/base8",
"@jupyter-widgets/controls": "@jupyter-widgets/controls8",
"@jupyter-widgets/jupyterlab-manager": "@jupyter-widgets/jupyterlab-manager8",
"@widgetti/solara-widget-manager": "@widgetti/solara-widget-manager8",
}
},
mode: 'development',
Expand Down
Loading

0 comments on commit 921c59d

Please sign in to comment.