-
Notifications
You must be signed in to change notification settings - Fork 25
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
Implement protocol 3.17.0 #10
Comments
Ok, I checked the changes, I think I can fire a PR by the end of week (if you don't see any complication I might have missed). Could you please assign it to me ? |
You are an angel, thank you! Will happily assign to you. |
I didn't forget you @tliron, still implementing the language server first before upgrading the protocol (if that's ok with you) |
👋 Hi, folks. Is there anything I can do to help move this forward? I'm a new user to this library and would love to use 3.17-specific types. Thanks! |
@semanticart Thank you for volunteering to submit a PR for 3.17 support. :) |
Good day everyone I'm looking on code of protocol_3_17 and as for me you are going hard way. As for me it will be easier to create separated repo with just protocol_3_16 and to fork it as protocol_3_17 (so any bug fixes from 3.16 can be merged to 3.17) and so on protocol_3_18... so any fixes to previous versions could be merged to last one. |
Hello,
First of all, thanks for the library.
It is very good, and I'm currently experimenting the creation of a language server based on it (and your puccini-language-server as a reference) 🙏
The spec just got upgraded a few days ago to 3.17.0 (05/10/2022), we should upgrade the protocol accordingly (if it's not too much effort).
I'm ready to help, still working around your code atm, but totally available 😇
The text was updated successfully, but these errors were encountered: