Skip to content

[V2] Whisper implementation#356

Open
InzeNL wants to merge 4 commits into
tmijs:old-v2from
InzeNL:v2-whisper_implementation
Open

[V2] Whisper implementation#356
InzeNL wants to merge 4 commits into
tmijs:old-v2from
InzeNL:v2-whisper_implementation

Conversation

@InzeNL

@InzeNL InzeNL commented Sep 16, 2019

Copy link
Copy Markdown

Implements the whisper command for TMI and adds tests for it

Also added codestyle files for JetBrains IDEAs to prevent overriding set codestyles

@InzeNL

InzeNL commented Sep 16, 2019

Copy link
Copy Markdown
Author

Requires implementation of WHISPER event

@InzeNL

InzeNL commented Sep 16, 2019

Copy link
Copy Markdown
Author

#356 (comment)

Requires implementation of WHISPER event

Implemented in 150b6e6

@Geczy

Geczy commented Nov 3, 2022

Copy link
Copy Markdown

can this be merged? whats the hold up @InzeNL

@InzeNL

InzeNL commented Nov 4, 2022

Copy link
Copy Markdown
Author

@Geczy

Granted I don't know how well this still works, there's no hold up on my side.
It was simply never reviewed by anyone.

I do notice this merges to "old-v2" so I'm not sure how valid the pull request is.

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