ConfigurationをNuxtConfigに変更する

[goki90210]

👏 解決する issue / Resolved Issues

• close @nuxt/types の Configuration が 非推奨になった #4923

📝 関連する issue / Related Issues

⛏ 変更内容 / Details of Changes

• @nuxt/typesのConfigurationがDeprecatedとなったためConfigurationをNuxtConfigに、Configuration as WebpackConfiguration を Configurationに変更する。
• extendの引数の型定義が不要となるため、
  import {Configuration as WebpackConfiguration} from 'webpack'
  を削除し、これにともないwebpackを直接参照する箇所がなくなるため、packege.jsonのdevDependenciesからwebpackを削除する。

📸 スクリーンショット / Screenshots

[munierujp]

NuxtConfig.build.extendに型がついているので、明示的に引数の型を指定する必要はなさそうです。

https://github.com/nuxt/nuxt.js/blob/05fe682f1db152ff9ade7dc9cc66435e19fa09d0/packages/types/config/build.d.ts#L144-L149

[goki90210]

@munierujp

NuxtConfig.build.extendに型がついているので、明示的に引数の型を指定する必要はなさそうです。

ありがとうございます。
extend(config: Configuration, _: any) {
を
extend(config: any, _: any) {
にしてもビルドは通りますね。
ただanyの省略はできません。
（_については使用していないのでanyで問題ない認識です。）

メリットとしては
extend(config: any, _: any) {
とすることで、webpackの参照がなくなってpackage.jsonから消しても問題がなくなるところでしょうか。
一長一短ですね。

[munierujp]

@goki90210 以下のようにすれば、configの型が推論されませんか？

extend(config) {

[goki90210]

@goki90210 以下のようにすれば、configの型が推論されませんか？

extend(config) {

@munierujp
確かに…。こうするとwebpackの直接の参照がなくなるのでpackage.jsonで縛らなくてもよくなりますね。
ありがとうございます。

[kaizumaki]

ご提案ありがとうございます！yarn.lockが激しくコンフリクトしますので、先に #4933 を採用する方向でいいでしょうか？

[goki90210]

@kaizumaki
わかりました。後で合わせます。

[kaizumaki]

LGTM!