Skip to content
This repository was archived by the owner on Feb 1, 2019. It is now read-only.

Conversation

@heyLu
Copy link
Contributor

@heyLu heyLu commented Jan 31, 2014

For that to work we wrap the code we get in :{ ... :} and pass that to ghci. If the code we got is a function binding we also wrap it in a let statement.

Should work at least on our codebase, haven't tried with anything bigger.

Before we merge this I'd like to add some tests and hear from others that it works for them (hopefully on slightly more complex projects).

@jetaggart
Copy link
Owner

Hey, I'm in the process of traveling across the world and heading back home. I'll be fairly unavailable (no laptop) for the next 3 weeks. Feel free to merge if you're comfortable with it. Sorry I can't be more helpful right now!

I'll try to look at this today, though.

@heyLu
Copy link
Contributor Author

heyLu commented Feb 12, 2014

Thanks for the heads up. I've not worked on this since I opened the pull request. I still plan to add some tests before merging this, though.

Do you have any preference for a Haskell testing library?

@jetaggart
Copy link
Owner

I like hspec with quickcheck, but don't care that much. Whatever you
think is best.

On Wed, Feb 12, 2014, at 04:05 AM, Lucas Stadler wrote:

Thanks for the heads up. I've not worked on this since I opened the pull
request. I still plan to add some tests before merging this, though.

Do you have any preference for a Haskell testing library?


Reply to this email directly or view it on GitHub:
#34 (comment)

heyLu added 3 commits October 4, 2014 19:33
doesn't support evaluating function definitions yet (coming soon). data
declarations work, though.
heyLu added a commit that referenced this pull request Oct 4, 2014
@heyLu heyLu merged commit 9066dd5 into master Oct 4, 2014
@heyLu
Copy link
Contributor Author

heyLu commented Oct 4, 2014

Ok, I think I won't get around to test this in the near future, so I'll just trust Haskell's "if it compiles, it will probably run well" magic fairy dust, the fact that travis compiles it fine and that it works locally.

@everyone: If you want to write tests, please do, I'd be happy to help you do that!

@heyLu heyLu deleted the multiline-eval branch October 4, 2014 19:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants