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

Add pict/code/eval with typeset-interaction, a la scribble/eval #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lexi-lambda
Copy link
Member

This adds a pict/code/eval module that provides interaction and typeset-interaction, which allows typesetting REPL interactions as picts, much like interaction from scribble/eval.

The way I've implemented it, this introduces a new dependency on scribble-lib to take advantage of make-base-eval. If this is problematic or if it should be avoided, it would be possible to implement it another way, instead (though it would likely involve implementing something very similar to make-base-eval, anyway).

@lexi-lambda lexi-lambda force-pushed the add-pict-code-eval branch 3 times, most recently from ed8d526 to 972e990 Compare September 13, 2015 07:22
@mflatt
Copy link
Member

mflatt commented Sep 17, 2015

I would like to avoid a dependency of pict-lib on scribble-lib. Should make-base-eval move to its own package? Or is it thin enough but useful enough to add (with a better name) to racket/sandbox?

@samth
Copy link
Member

samth commented Sep 17, 2015

I don't particularly care about this case, but in general I think we should aim for more, smaller, packages rather fewer larger ones. This approach has worked very well in a number of other language communities, and I think it would be good in Racket as well.

@lexi-lambda
Copy link
Member Author

I'd be fine with moving it into racket/sandbox or moving it into its own package. Do you have a particular preference?

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