-
-
Notifications
You must be signed in to change notification settings - Fork 683
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
Adding ongaku to the client.md list. #1017
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps this library should be listed further up with the other purely Python libraries?
oh yeah sorry, didn't know how it was ordered 😅 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I am doing this right, this should? have my fix, however, I am unsure.
Edit: I have no idea how to get my changed version of this into this pr sorry. I did move it down so it was alphabetical, however, I cannot figure out how to get it to show up here.
I am as confused as you are. It looks like you pushed to the correct branch in your fork, but it isn't reflecting here. |
|
lmao, what a coincidence. I even googled to make sure I was doing the right thing, and it's a github issue xD |
on that note: you should move it to the bottom of the python clients |
I'll be able to do that tomorrow if I do remember. Otherwise, you are more than welcome to move it and merge the branch as you please. |
okay, I have re-updated it to add mine at the bottom of the python list. |
Adding my client to the list of clients on the Lavalink website.