Skip to content

Commit

Permalink
build: webpack ts-loader transpileOnly set to true
Browse files Browse the repository at this point in the history
Needed this because otherwise the perfectly valid Typescript code would
not compile when using TS loader in cases when multiple Cactus packages
were depending on types from certain dependencies (example both core-api
and the besu connector would depend on the socketio package and its
"Server" class would be claimed as incompatible by the TS loader
compilation process even when the source files for those types were byte
to byte the same (but the path was different to them because they
existed in different node_modules directories)

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
  • Loading branch information
petermetz committed May 19, 2021
1 parent 599acc0 commit 67d2e79
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion webpack.dev.node.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ module.exports = {
{
loader: "ts-loader",
options: {
transpileOnly: false,
transpileOnly: true,
configFile: "tsconfig.json",
},
},
Expand Down
2 changes: 1 addition & 1 deletion webpack.dev.web.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ module.exports = {
{
loader: "ts-loader",
options: {
transpileOnly: false,
transpileOnly: true,
configFile: "tsconfig.json",
},
},
Expand Down
2 changes: 1 addition & 1 deletion webpack.prod.node.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ module.exports = {
{
loader: "ts-loader",
options: {
transpileOnly: false,
transpileOnly: true,
configFile: "tsconfig.json",
},
},
Expand Down
2 changes: 1 addition & 1 deletion webpack.prod.web.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ module.exports = {
{
loader: "ts-loader",
options: {
transpileOnly: false,
transpileOnly: true,
configFile: "tsconfig.json",
},
},
Expand Down

0 comments on commit 67d2e79

Please sign in to comment.