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 a stdio based Janet client #536

Merged
merged 1 commit into from
Dec 3, 2023
Merged

Conversation

sogaiu
Copy link

@sogaiu sogaiu commented Oct 27, 2023

This is a basic stdio client for Janet based on some of the existing stdio clients.

There is already a nice netrepl-based client for Janet, though that requires one to install and setup a netrepl server for use.

A stdio-based client allows one to avoid the installation of both jpm and spork (possibly a speed bump for folks who are in the process of evaluating [1] Janet) but also might be nice for quick interactive sessions. As with other stdio-based solutions though, functionality is reduced. Perhaps a worthwhile trade-off for some situations.

This PR includes document updates and additions but the Wiki has not been updated yet.


[1] Pun intended :P

@sogaiu sogaiu changed the base branch from master to develop October 27, 2023 01:13
@sogaiu
Copy link
Author

sogaiu commented Oct 27, 2023

Sorry, I failed to choose the develop branch when creating this PR initially. Have made an attempt to correct that.

@Olical Olical merged commit 5036ec7 into Olical:develop Dec 3, 2023
2 checks passed
@Olical
Copy link
Owner

Olical commented Dec 3, 2023

This is really cool! Just tried it out and it worked great for me! I'd even consider making this the default because I love the "it just works" approach but that might break some users' workflows, so maybe not just yet. Thank you so much for putting all of the effort into this and I'm sorry it's taken so long to merge it, I've just been really busy with life, work and OSS, I must've lost this notification in my inbox 😢

Merged now though! Will release ASAP as soon as some other issues are confirmed to be fixed / addressed by other people.

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.

None yet

2 participants