インポート構文の統一

[acid-chicken]

Summary

現状

https://github.com/syuilo/misskey/blob/125849673a1aba46021852ee473d00f4520d1bd6/src/server/web/docs.ts#L5-L20

---

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

[AyaMorisawa]

どのように統一するんですか？

[acid-chicken]

とりあえずCommonJS構文は可能な限りES6構文に寄せるなりして統一した方が良いかと。

[mei23]

https://github.com/syuilo/misskey/pull/4095#pullrequestreview-199348636

import ms = require('ms');
はTypeScript独自の構文でCommonJSでもES6でもなさそう
https://www.typescriptlang.org/docs/handbook/modules.html の export = and import = require()

なので、CommonJS / ES6 が混在しているわけではない

Visual Studio Codeで自動インポートがimport ms = require('ms');になるのは
おそらく型定義がexport = ms;になっているからで

When exporting a module using export =, TypeScript-specific import module = require("module") must be used to import the module.

ともあるので
このような場合はimport ms = require('ms');の方法を使わなければならないのではと思います。

[acid-chicken]

型定義の書き方によって複数の import の書き方を課せられるぐらいなら型定義を変えた方が良さそうと思ってしまうのですが、どう思われますか。

[mei23]

import * as ms from 'ms'; => import ms = require('ms'); に統一される形なので
複数のimportの書き方を強要されているわけではないと思います

[acid-chicken]

そっちに統一する感じですか。

[mei23]

import * as ms from 'ms'; => import ms = require('ms'); に統一される形なので

でも、module exportsにはimport * as ... from使えみたいに見えるから統一してもダメなのかも

[mei23]

とりえあず、CommonJS と ES6 が混在しているわけではないのでどちらも許容でいいのではと
(const ... requireの構文を廃止するという目的でいいのでは)

[acid-chicken]

だいたい統一できた。

[tamaina]

CONTRIBUTING.mdに方針を書いておいてもらえると有難いかなと。

[KisaragiEffective]

const ident = require(moduleSpecifier)
を禁止することをCONTRIBUTING.mdに書いておいたらいいですか？