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

Collaboration with CIDER's team #2

Open
bbatsov opened this issue Sep 27, 2018 · 2 comments
Open

Collaboration with CIDER's team #2

bbatsov opened this issue Sep 27, 2018 · 2 comments

Comments

@bbatsov
Copy link

bbatsov commented Sep 27, 2018

Hey there!

vim-iced looks like an awesome project! Congratulations for creating it!

I'm the author of CIDER and I noticed that you've created some extra middleware for vim-iced, so I just wanted to let you know that our team really wants to have orchard, cider-nrepl and refactor-nrepl be useful to as many editors as possible, so if you feel some of the functionality you've developed is generic enough we'd be glad to have it there.

I'm not sure about the rationale behind the different ops you've added so far, but some seem like something that we can definitely use in CIDER as well (e.g. the grimoire stuff, the linter stuff, etc).

Btw, I was under the impression there's already some op in cider-nrepl to return a list of the project namespaces. There are also ops about formatting the code with cljfmt, so I'd be curious to hear why you couldn't reuse those.

Anyway, just wanted to let you know I'm excited to see another person working on nREPL middleware and that me and the rest of the CIDER/nREPL team are here to help and support you. It'd be great if there was more collaboration between the various tool authors. :-)

@liquidz
Copy link
Owner

liquidz commented Sep 27, 2018

@bbatsov
vim-iced is still alpha version, and iced-nrepl is a experimental project to try function I want to use with vim-iced.

so if you feel some of the functionality you've developed is generic enough we'd be glad to have it there.

I'm happy to hear that!
I would like to contribute some codes what I think useful to cider/refactor-nrepl.

Btw, I was under the impression there's already some op in cider-nrepl to return a list of the project namespaces. There are also ops about formatting the code with cljfmt, so I'd be curious to hear why you couldn't reuse those.

I couldn't find the op that returns orchard.namespace/project-namespaces simply.
ns-list seems to return all loaded namespaces.

format-code op is not accepting options for cljfmt.core/reformat-string.
https://github.com/weavejester/cljfmt/blob/0338e08daf8b6ede3cd1a97a09ba5d660ef0f6ef/cljfmt/src/cljfmt/core.cljc#L384
Especially, I would like to specify :indents for indent customization.

@bbatsov
Copy link
Author

bbatsov commented Oct 1, 2018

I couldn't find the op that returns orchard.namespace/project-namespaces simply.
ns-list seems to return all loaded namespaces.

Hmm, I guess that's some oversight on our end then. We should add this to cider-nrepl.

Especially, I would like to specify :indents for indent customization.

That's a good addition. Feel free to submit this upstream.

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

No branches or pull requests

2 participants