search

diraq-sea / diraq_desktop

🎛 DiraQ desktop application
1 stars 0 forks source link

Fix mock file loading script #116

Closed hi85gh closed 5 years ago

hi85gh commented 5 years ago

main/mocks/models ディレクトリに .DS_Store など JavaScript 以外のファイルが入ってしまった場合にエラーになってしまうため、 filter() メソッドを追加しました。

hi85gh commented 5 years ago

Prettier のみ VS Code のエクステンションとバージョンが合わず、オートフォーマットでエラーが出てしまうためアップデートは見送っています。

111 で上記の報告をしていましたが、今の Prettier のバージョンでも以下のようにオートフォーマットされてしまい、エラーが発生していました。

      await Promise.all(
        fs.readdirSync(path.join(__dirname, '../mocks/models')).map(filename =>
          (async () => {
-             mock[
-               filename.replace('.js', '')
-             ] = (await import(`../mocks/models/${filename}`)).default.defaultValues()
+             mock[filename.replace('.js', '')] = (await import(
+               `../mocks/models/${filename}`
+             )).default.defaultValues()
          })(),
        ),
      )

今回の修正で解消されているかと思います。

hi85gh commented 5 years ago

111 で上記の報告をしていましたが、今の Prettier のバージョンでも以下のようにオートフォーマットされてしまい、エラーが発生していました。

作業当日に VS Code のエクステンションが v1.9.0 にアップデートされていたので差分が出ていたようです。

hi85gh commented 5 years ago

すみません、ログイン後にエラーが出ていたのであらためて修正します。

hi85gh commented 5 years ago

@m-mitsuhide
修正に加えて以下の変更を加えました。 ご確認をお願いします。