The Wayback Machine - https://web.archive.org/web/20201229132210/https://github.com/atom/language-perl/pull/58
Skip to content
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

Improve regexp highlighting #58

Merged
merged 1 commit into from Dec 4, 2015
Merged

Conversation

@Fank
Copy link
Member

@Fank Fank commented Nov 19, 2015

Fixes #53

@50Wliu
Copy link
Member

@50Wliu 50Wliu commented Dec 1, 2015

Looks good to me 😄.

@@ -637,7 +637,7 @@
]
}
{
'begin': '(?<=\\(|\\{|~|&)\\s*((\\/))'
'begin': '(?<=\\(|\\{|~|&|\\||if|unless|^)\\s*((\\/))'

This comment has been minimized.

@50Wliu

50Wliu Dec 1, 2015
Member

This is obviously something really small and definitely doesn't need to block this PR, but the curly bracket { doesn't need to be escaped. Also, I think ^ is usually placed at the beginning of the match.

kevinsawicki added a commit that referenced this pull request Dec 4, 2015
Improve regexp highlighting
@kevinsawicki kevinsawicki merged commit 9dec4fd into master Dec 4, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@kevinsawicki
Copy link
Member

@kevinsawicki kevinsawicki commented Dec 4, 2015

Thanks for this @Fank

@kevinsawicki kevinsawicki deleted the fk_regex_find_improvements branch Dec 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.