Skip to content

Conversation

Malabarba
Copy link
Member

This PR adds support for the Enlighten feature in clojure-emacs/cider-nrepl#296.

Take a file like the following:

screenshot-2016-02-05--053810

Turning on cider-enlighten-mode and evaluating the sexps one by one (C-c C-k doesn't work) will lead to this:

screenshot-2016-02-05--053747

@bbatsov
Copy link
Member

bbatsov commented Feb 5, 2016

You've just signed LT's death sentence. :-) The code looks good - all we need now is some documentation and a changelog entry. You can add this (or an animated gif) in the README.

@malchmih
Copy link

malchmih commented Feb 5, 2016

Simply awesome! 👍

@arrdem
Copy link
Contributor

arrdem commented Feb 5, 2016

Sweet!

@Malabarba
Copy link
Member Author

Thanks. :-)

all we need now is some documentation and a changelog entry.

Yeah, I'll start writing that now.

@Malabarba Malabarba force-pushed the enlighten branch 2 times, most recently from 287f8f5 to 5d5653c Compare February 5, 2016 09:04
@bbatsov
Copy link
Member

bbatsov commented Feb 5, 2016

The docs look good. Guess you can squash the commits and we can merge this.

README.md Outdated
(setq cider-stacktrace-fill-column 80)
```

### Enlighten (Light Table)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Btw, maybe the reference to "Light Table" will be big misleading, as most people will think this is something like "InstaREPL".

Handle :enlighten messages through the debug channel.
Define cider-enlighten-mode.
Font-lock enlightened defns.
Document Enlighten.
@Malabarba Malabarba force-pushed the enlighten branch 2 times, most recently from 24ba6f6 to c4e55e2 Compare February 5, 2016 22:20
Malabarba added a commit that referenced this pull request Feb 5, 2016
@Malabarba Malabarba merged commit 66826d0 into master Feb 5, 2016
@Malabarba Malabarba deleted the enlighten branch February 5, 2016 22:20
@crinklywrappr
Copy link

This is awesome!

@Malabarba
Copy link
Member Author

😊

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.

5 participants