Skip to content

tune(monaco-editor-webpack-plugin): expose plugin options type#2853

Merged
alexdima merged 1 commit intomicrosoft:mainfrom
ValeraS:webpack-plugin
Jan 14, 2022
Merged

tune(monaco-editor-webpack-plugin): expose plugin options type#2853
alexdima merged 1 commit intomicrosoft:mainfrom
ValeraS:webpack-plugin

Conversation

@ValeraS
Copy link
Contributor

@ValeraS ValeraS commented Dec 24, 2021

Our build tools use this plugin under the hood, and its options are part of a larger configuration. So it would be great to get the types directly from this plugin.

@hediet
Copy link
Member

hediet commented Jan 3, 2022

Thanks for this PR! I don't know the semantics of declare namespace by heart. What is the advantage of this over exporting the interface type?

@ValeraS
Copy link
Contributor Author

ValeraS commented Jan 3, 2022

When using export = something, we cannot export anything else from this module.

@alexdima
Copy link
Member

Nice trick (to use a namespace with the same name as the class)!

@alexdima alexdima merged commit c9e1c99 into microsoft:main Jan 14, 2022
@alexdima alexdima added this to the January 2022 milestone Jan 14, 2022
@ValeraS ValeraS deleted the webpack-plugin branch January 14, 2022 14:54
@github-actions github-actions bot locked and limited conversation to collaborators Feb 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants