Rename submodule paths to fix Dependabot submodule updates

[victor-gp]

Fixes #3198.

Before:

• Failing Dependabot submodules job
• Fails because it cannot locate the Javascript (Babel) submodule (responds 302 then 404):
  https://api.github.com/repos/sharkdp/bat/contents/assets/syntaxes/02_Extra/JavaScript%20(Babel)?ref=4fc55cfcec44dba3f67bd79cac9ccf64dc18eb08
• Still failing when the ref is master:
  https://api.github.com/repos/sharkdp/bat/contents/assets/syntaxes/02_Extra/JavaScript%20(Babel)?ref=master

After renaming to Javascript_(Babel):

• Request for the submodule succeeds (responds 302 then 200):
  https://api.github.com/repos/victor-gp/bat/contents/assets/syntaxes/02_Extra/JavaScript_(Babel)/?ref\=fix-submodules-for-dependabot

After merging to master:

• The request for the submodule should succeed (302 => 200):
  https://api.github.com/repos/sharkdp/bat/contents/assets/syntaxes/02_Extra/JavaScript_(Babel)/?ref\=master
• Succesful Dependabot submodules job? 🤞

[victor-gp]

I've noticed that these submodules have a .sublime-syntax counterpart, a file owned by bat in the same syntaxes/02_Extras/ directory as the submodule. [^1]

I've also noticed that the repositories the submodules point to don't have any sublime-syntax file, they have .tmLanguage files and similar. So I gather that the owned file is generated somehow from the submodule's own?

In such case, should we also rename those files?

[1] All 4 changed submodules, e.g.:

• https://github.com/sharkdp/bat/tree/master/assets/syntaxes/02_Extra#:~:text=JavaScript%20(Babel)%20%40%20f4579f9
• https://github.com/sharkdp/bat/blob/master/assets/syntaxes/02_Extra/JavaScript%20(Babel).sublime-syntax

[keith-hall]

I've also noticed that the repositories the submodules point to don't have any sublime-syntax file, they have .tmLanguage files and similar. So I gather that the owned file is generated somehow from the submodule's own?

Yes, we have this documented here: https://github.com/sharkdp/bat/blob/master/doc/assets.md#adding-new-builtin-languages-for-syntax-highlighting

In such case, should we also rename those files?

I would say no, because unless the file contains a name key inside it's yaml contents, the base file name is the name given to the syntax definition and used when mapping syntaxes, setting language from the command line arg etc. So we don't want to change that 🙂

[keith-hall]

Thanks!