-
Notifications
You must be signed in to change notification settings - Fork 128
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
Vim Script and Vim script should be combined #115
Comments
I see, thanks for reporting, will be fixed soon. |
I added a fix that filters the lowercase "Vim script" and leaves "Vim Script." I think "Vim Script" is the official name on GitHub, and I'm not sure why there is also a lowercase variant in the GitHub BigQuery Dataset. Currently, I do not have enough credits/dollars left to rerun the entire history with a proper merge in SQL. Therefore, I filtered out the lowercase naming, which also has a significantly lower count of events. |
The official name is "Vim script" as per the manual. And Wikipedia: https://en.wikipedia.org/wiki/Vim_(text_editor)#Vim_script If you ever do a re-run, I would change it to that. I would not filter out "Vim script", because this results in an inaccurate comparison of languages. Having a duplicate is preferable to this in my opinion. |
You are right, but I referred to the "official name on GitHub". For instance if you open one of the most popular Vim script projects on GitHub and look at the language name https://github.com/tpope/vim-fugitive it will read "Vim Script". |
Ah right, so maybe we should file a bug report with GitHub. |
No description provided.
The text was updated successfully, but these errors were encountered: