You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The script for adding custom hotkeys breaks if a repo owner partially matches any of Github's reserved names. For example, hotkeys don't work on git-for-windows/git because it has 'windows' in the owner's name.
This seems to be caused by JS testing a regex at all positions in a given string instead of only at the start (even without a global flag):
The script for adding custom hotkeys breaks if a repo owner partially matches any of Github's reserved names. For example, hotkeys don't work on git-for-windows/git because it has 'windows' in the owner's name.
This seems to be caused by JS testing a regex at all positions in a given string instead of only at the start (even without a global flag):
GitHub-userscripts/github-custom-hotkeys.user.js
Lines 56 to 80 in a591264
GitHub-userscripts/github-custom-hotkeys.user.js
Lines 98 to 102 in a591264
I think this issue can be safely fixed by surrounding the regex with
^
and$
.P.S. Also it seems like the
{me}
scope is always undefined:GitHub-userscripts/github-custom-hotkeys.user.js
Lines 91 to 93 in a591264
The text was updated successfully, but these errors were encountered: