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

Atom editor support for Mal is poor #562

Open
wgmyers opened this issue May 27, 2021 · 7 comments
Open

Atom editor support for Mal is poor #562

wgmyers opened this issue May 27, 2021 · 7 comments

Comments

@wgmyers
Copy link

wgmyers commented May 27, 2021

I should know: such minimal support as exists - until now there wasn't any at all - is all my fault. Apologies in advance.

So fwiw I have implemented basic Atom support for Mal: https://atom.io/packages/language-lisp-mal

(Sadly, language-mal was taken, hence language-lisp-mal.)

In the unlikely event that the Venn diagram of people playing around with Mal and Atom users consists of anyone other than just me, this may be of interest, for certain values of 'interest'. Keywords, string constants and integers are supported. Reader macros not yet. User-defined anything, not really, except for the first instance of the thing defined by def! or defmacro!.

As an aside, implementing a syntax highlighting package for Mal was several orders of magnitude less fun than implementing Mal was.

Bug reports and pull requests welcome.

I'm about yay close to going back to Emacs at this point.

@sogaiu
Copy link

sogaiu commented May 27, 2021

I don't know if you are interested in using tree-sitter, but FWIW, there is this: https://github.com/sogaiu/tree-sitter-mal

@wgmyers
Copy link
Author

wgmyers commented May 27, 2021

Thank you! That would certainly take the pain out of making a tree-sitter version: having to write a tree-sitter grammar first before going ahead with the syntax highlighter was why I chose to go the TextMate route.

@asarhaddon
Copy link
Contributor

Please rename this issue. There is an atom type in MAL with deliberately limited support, so the current title is confusing.

@wgmyers wgmyers changed the title Atom support for Mal is poor Atom editor support for Mal is poor May 29, 2021
@wgmyers
Copy link
Author

wgmyers commented May 29, 2021

Apologies. Done.

@asarhaddon
Copy link
Contributor

Apologies in return, then. Your answer makes me realize that my tone was crude, please be indulgent to my english skills and have fun with MAL !

@wgmyers
Copy link
Author

wgmyers commented May 30, 2021

Not at all, no need to apologise. Thank you for pointing out the ambiguity - if I have been struggling to write anything useful in Mal, it is in part because I keep forgetting about Mal atoms...

@kanaka
Copy link
Owner

kanaka commented May 31, 2021

@wgmyers nice! Please send a PR with a modification to the README to add a link and brief description of this (probably to the "Other mal Projects" section).

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

No branches or pull requests

4 participants