-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
Switch VB language to different fork #147690
base: main
Are you sure you want to change the base?
Conversation
The original project is not maintained [anymore](textmate/asp.vb.net.tmbundle#11) so I decide to take maintainance. Not sure if this is proper place, but if yes, then I hope during next sync commit kant2002/asp.vb.net.tmbundle@5164486 will land in this repo.
@kant2002 thanks for your interest in improving the VB syntax highlighting support! We have a number of grammars that have been abandoned like this, so it's always great when someone offers a better maintained one. I usually prefer to see some longer running support and use before switching to a new repo for a grammar. Would you be amenable to creating a VS Code extension with your grammar? You would be able to copy a lot of the built-in VB extension's package.json. Then, after it's been in the wild for a month or two we can revisit this switch. |
@alexr00 that totally make sense. Can you point me of similar single purpose extension? Or should I just follow tutorial for new language extension and that's all what I need? |
Other problem which I do not know how to solve is marketing. I do not have clout in VB community, and made fix because it affects me. and because VB syntax works mostly out of the box, people would not even look for other alternatives. So I do not expect that my small change would be hugely important. Also consider that people who work with VS Code tolerate these small errors much better then IDE users because they have lower expectations to VS code. |
The tutorial for a new language extension should get you started well.
@kant2002 do you plan to make additional improvements to the grammar? |
Thanks, thats fine. Now I understand what you are proposing.
I do have to find additional issues with it,. I definitely want to make things working for VB, but I cannot predict what currently is broken (hopefully not much) |
@kant2002, I'm going to close this as part of our housekeeping since there hasn't been any activity in a while, but if this is still something you want to work on let me know! |
@alexr00 I definitely want to have VB grammar to be fixed. So if you can, please reopen |
The original project is not maintained anymore so I decide to take maintenance.
Not sure if this is proper place to change, but if yes, then I hope during next sync commit kant2002/asp.vb.net.tmbundle@5164486 will land in this repo.