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

True,false, nil are highlighted as atoms #244

Merged
merged 2 commits into from
Sep 3, 2015

Conversation

edmz
Copy link
Contributor

@edmz edmz commented Sep 3, 2015

No description provided.

@mattdeboard
Copy link
Contributor

👍 for including tests

@edmz
Copy link
Contributor Author

edmz commented Sep 3, 2015

to be honest, @tonini made me do it :P

I had to understand how they worked

On Thu, Sep 3, 2015 at 12:25 PM Matt DeBoard notifications@github.com
wrote:

[image: 👍] for including tests


Reply to this email directly or view it on GitHub
#244 (comment)
.

@mattdeboard
Copy link
Contributor

Well then I revoke my 👍 and award it to @tonini

mattdeboard added a commit that referenced this pull request Sep 3, 2015
True,false, nil are highlighted as atoms
@mattdeboard mattdeboard merged commit f899bc7 into elixir-editors:master Sep 3, 2015
@mattdeboard
Copy link
Contributor

May God have mercy on your soul

@tonini
Copy link
Contributor

tonini commented Sep 3, 2015

😆

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.

3 participants