Skip to content

[karma-webpack] Fix build by adding type for Logger#22572

Merged
1 commit merged intoDefinitelyTyped:masterfrom
Flarna:karma-webpack_fix_build
Jan 2, 2018
Merged

[karma-webpack] Fix build by adding type for Logger#22572
1 commit merged intoDefinitelyTyped:masterfrom
Flarna:karma-webpack_fix_build

Conversation

@Flarna
Copy link
Copy Markdown
Contributor

@Flarna Flarna commented Dec 29, 2017

Got following build error:

Error in karma-webpack
Command failed: node /home/travis/build/DefinitelyTyped/DefinitelyTyped/node_modules/dtslint/bin/index.js --onlyTestTsNext
Error: /home/travis/build/DefinitelyTyped/DefinitelyTyped/types/karma-webpack/index.d.ts
ERROR: 29:36  expect  TypeScript@next compile error: 
Namespace 'WebpackDevMiddleware' has no exported member 'Logger'.
ERROR: 30:37  expect  TypeScript@next compile error: 
Namespace 'WebpackDevMiddleware' has no exported member 'Logger'.
ERROR: 31:38  expect  TypeScript@next compile error: 
Namespace 'WebpackDevMiddleware' has no exported member 'Logger'.

I think it's caused by #22267.

karma-webpack is still using 1.12.x of webpack-dev-middleware therefore I added only a local definition of Logger to fix this as there is no v1 typing for webpack-dev-middleware.

@ghost ghost merged commit 39f18d5 into DefinitelyTyped:master Jan 2, 2018
@Flarna Flarna deleted the karma-webpack_fix_build branch January 2, 2018 18:34
This pull request was closed.
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.

1 participant