Skip to content

Revert to the version 2.0.0 'foo.exe' reconginition. Fix for #87 and #95 #96

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

Merged
merged 1 commit into from
Feb 26, 2015
Merged

Conversation

vors
Copy link
Member

@vors vors commented Feb 26, 2015

I cannot figure out why it broke interpolation, but this is a pretty unusual syntax. Let's avoid it.

…out why it broke interpolation, but this is a pretty unusual syntax. Let's avoid it. Fix for #87 and #95
@vors
Copy link
Member Author

vors commented Feb 26, 2015

@bcrotty, please, take a look.
I think sublime went crazy because of

(?i:([\S&&[^<>"/\|?*]])+)

@bcrotty
Copy link
Contributor

bcrotty commented Feb 26, 2015

Maybe this is what's broken with #100. Let's revert this, and I'll try to figure out some alternate ways of doing this.

vors added a commit that referenced this pull request Feb 26, 2015
Revert to the version 2.0.0 'foo.exe' reconginition. Fix for #87 and #95
@vors vors merged commit 7cd8802 into SublimeText:dev Feb 26, 2015
@vors vors deleted the bugfixes branch March 2, 2015 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants