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 some racket support #488

Closed
wants to merge 1 commit into from
Closed

Conversation

odanoburu
Copy link
Contributor

  • submitting input to racket REPL now works out-of-the-box
  • go-to definition works

added some definitions to racket-mode too, see greghendershott/racket-mode#374

- submitting input to racket REPL now works out-of-the-box
- go-to definition works
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 62.769% when pulling d0ff772 on odanoburu:racket into 27282ca on abo-abo:master.

@abo-abo abo-abo closed this in a9ebdca May 21, 2019
abo-abo added a commit that referenced this pull request May 21, 2019
@abo-abo
Copy link
Owner

abo-abo commented May 21, 2019

Thanks. Merged with minor changes, please review.

I also added some limited support for eval. It doesn't display any stdout, only the return value. But better than nothing.

I wonder, why not just use Geiser? Is racket-repl better?

@odanoburu
Copy link
Contributor Author

I also added some limited support for eval.

thanks! I saw some people wanted this feature in racket-mode's issues.

I wonder, why not just use Geiser?

tbh, I haven't even tried it. because racket is not scheme anymore i thought things might break :P

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