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

Connect to nrepl by URL #328

Closed
RomanTsopin opened this issue Sep 19, 2019 · 5 comments
Closed

Connect to nrepl by URL #328

RomanTsopin opened this issue Sep 19, 2019 · 5 comments

Comments

@RomanTsopin
Copy link

@RomanTsopin RomanTsopin commented Sep 19, 2019

There was a handy option in the previous version of Calva to connect to any nrepl by url without creating a project folder first. It was especially cool for lein try plugin to pull any dependency and start messing with it. Seems like it’s not working now.

@PEZ

This comment has been minimized.

Copy link
Collaborator

@PEZ PEZ commented Sep 20, 2019

Hello. This was a casualty in the work with bringing Jack-in and Connect into projects to a better position. I didn't think about this use case at all when we did that work...

I think it would be great if we make Calva support this again. I'll have a look at how difficult it would be to bring it back. (I don't think it should be that much trouble.)

PEZ added a commit that referenced this issue Sep 21, 2019
Addressing #328
@PEZ

This comment has been minimized.

Copy link
Collaborator

@PEZ PEZ commented Sep 21, 2019

@RomanTsopin , we have now added a separate command to connect to a non-project REPL, can you try this VSIX package out, and see that ut fixes the problem?

https://343-125431277-gh.circle-artifacts.com/0/tmp/artifacts/calva-2.0.40-wip-connect-non-project-repl-49d232f0.vsix

@RomanTsopin

This comment has been minimized.

Copy link
Author

@RomanTsopin RomanTsopin commented Sep 22, 2019

Thanks, works perfect!
Would be nice to have this option on the bottom nREPL icon. And if you ask me – disconnect (sometimes connection hangs) and restart (after deps are changed) commands too :) At least in command palette.

@PEZ

This comment has been minimized.

Copy link
Collaborator

@PEZ PEZ commented Sep 22, 2019

This command should be added to the nREPL icon... As for what more should be there, chip in on this PR: #333

@PEZ

This comment has been minimized.

Copy link
Collaborator

@PEZ PEZ commented Sep 25, 2019

This is now released.

@PEZ PEZ closed this Sep 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.