Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(webpack): fix broken bundling - cannot find webpack.config.js #2424

Merged

Conversation

deepto98
Copy link
Contributor

The examples/cactus-workshop-examples-2022-11-14 package was causing yarn build:dev to fail. This PR fixes the incorrect webpack config path in the example's package.json file and adds a missing /src/main/typescript/index.web.ts file.
Fixes https://github.com/hyperledger/cacti/issues/2357
Signed-off-by: deepto98 ddepp98@outlook.com

Copy link
Contributor

@jagpreetsinghsasan jagpreetsinghsasan left a comment

Choose a reason for hiding this comment

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

LGTM
Thankyou for the PR!

@sandeepnRES sandeepnRES force-pushed the fix-cannot-find-webpack.config.js branch 2 times, most recently from 9319570 to 760c778 Compare June 1, 2023 10:11
@sandeepnRES sandeepnRES enabled auto-merge (rebase) June 1, 2023 10:12
@ryjones ryjones disabled auto-merge June 1, 2023 14:30
Copy link
Contributor

@petermetz petermetz left a comment

Choose a reason for hiding this comment

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

LGTM

@petermetz petermetz force-pushed the fix-cannot-find-webpack.config.js branch from 760c778 to fe51632 Compare June 14, 2023 04:21
@petermetz petermetz enabled auto-merge (rebase) June 14, 2023 04:22
@petermetz petermetz force-pushed the fix-cannot-find-webpack.config.js branch from fe51632 to f239430 Compare June 14, 2023 13:41
@petermetz petermetz merged commit 794f0b4 into hyperledger-cacti:main Jun 14, 2023
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.

build(webpack): fix broken bundling - cannot find webpack.config.js
4 participants