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

おめでとうございます!あなたはクソデカお気持ちみたいなPRを受け取りました! #3

Merged
merged 12 commits into from
Dec 3, 2023

Conversation

Nich87
Copy link
Collaborator

@Nich87 Nich87 commented Dec 3, 2023

6fb09fb │ ciのブランチ名が誤っていたためmainに修正
02fc2f3queue.tsからYTPlayerクラスをplayer.tsとして切り出し
a2e0e4d │ 音量操作を出来るようにresourceをプロパティとして追加し、
     Queueクラスにlengthゲッターを作成することでわざわざstoresにアクセスする手間を無くしました。
ef93b21volumeコマンドとnowplayingコマンドを作成しました。
9165a0d │ コマンドハンドラを作成しsrc/index.tsが冗長にならないようにしました。
d4aa4e4 │ 型をつけました(間違ってたらすみません)
ea7f568 │ コマンドのエイリアスを設定しました。
08b00a9Queueクラスからジェネリクスを削除し、オプショナルチェーンや冗長なコードを修正しました。
906f901 │ 依存関係のアップデートを行いました。
b2670fbpnpm-lock.yamlのタイポです...後で修正します><
e6ffc3b...95510db │ eslintを適切に設定しました(順序が狂ってました)

@MotiCAT MotiCAT added Type: improvement 改善に関するPR Type: new feature 新機能に関するPR Type: Refactoring リファクタリングに関するPR labels Dec 3, 2023
@MotiCAT
Copy link
Owner

MotiCAT commented Dec 3, 2023

@Nich87 PR作成ありがとうございます。
b2670fb のタイポはこちらの方で修正しておきます:)
レビューが完了したのでマージいたします。

@MotiCAT MotiCAT merged commit b098984 into MotiCAT:main Dec 3, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: improvement 改善に関するPR Type: new feature 新機能に関するPR Type: Refactoring リファクタリングに関するPR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants