Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

npm run build時のコンパイルエラーに対処 #140

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

takejohn
Copy link

@takejohn takejohn commented Mar 24, 2024

npm run build時のコンパイルエラーをなくします。

関連 issue

変更点

  • config.ts で設定の型チェックを行う
  • 藍#api メソッドに ReturnType 型パラメータを作成して呼び出し側で ReturnType に型を与える
  • Message#reply のオーバーロードを定義して text の型が string の場合は返値の型を Promise<Note> となるように変更
  • Module の未初期化プロパティに関するエラーに対処
    • Module#install メソッドの引数に を追加
    • ai プロパティが インスタンスであることが求められる InstalledModule 抽象クラスを定義
    • Module における ai プロパティと関連メソッドの使用をJSDoc (TypeDoc?) にて非推奨としてタグ付け
    • check-custom-emojis, emoji-react, guessing-game, kazutori, keyword, reminder, server モジュールにおいて InstalledModule を使用
  • リバーシのバックエンド Session クラスにおいて、account, engine, botColor をゲッターにして、期待した型でなければエラーを送出
  • の未初期化プロパティに関するエラーに対処
    • コンストラクタの機能を start スタティックメソッドに、run インスタンスメソッドの機能をコンストラクタに移動
    • index.ts で のコンストラクタを呼び出す代わりに 藍.start メソッドを呼び出し

@takejohn takejohn marked this pull request as ready for review March 30, 2024 11:13
@takejohn
Copy link
Author

config.json がないとコンパイルエラーが出てしまうので直します

@takejohn takejohn marked this pull request as draft March 31, 2024 08:57
@takejohn takejohn marked this pull request as ready for review April 3, 2024 13:42
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.

2 participants