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

Fix error stemming from bitwise regex #3

Merged
merged 2 commits into from Mar 1, 2014

Conversation

rjsamson
Copy link
Contributor

@rjsamson rjsamson commented Mar 1, 2014

This PR fixes the issues causing the converted bundle to not work properly with Atom.

Possibly a display only bug in Atom, but the multiple / nested escapes
in the single quoted string were causing it to appear in editor as if
the string was not terminated properly. Replaced with heredoc.
On conversion from text mate bundle, apm appears to not be escaping `^`
in the regex.
lucasmazza added a commit that referenced this pull request Mar 1, 2014
Fix error stemming from bitwise regex
@lucasmazza lucasmazza merged commit f6ad867 into elixir-editors:master Mar 1, 2014
@lucasmazza
Copy link
Contributor

❤️ 💚 💙 💛 💜

@jonathanmarvens
Copy link
Contributor

@lucasmazza You forgot 💖.
@rjsamson This is for you: 💝.

@patrickdet
Copy link

thanks! awesome

@rjsamson
Copy link
Contributor Author

rjsamson commented Mar 1, 2014

@lucasmazza @jonathanmarvens thanks! Glad I could help.

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.

None yet

4 participants