Charles Comstock

Results 46 comments of Charles Comstock

I agree that case would be magical and I'm game to incrementally improve in that direction. However, I want to clarify the most common use case for this is not...

Agreed that we can reliably insert a reader conditional if it's in the project already, but AFAIK the middleware does not currently expose enough information to reliably suggest that to...

I thought I would add another example where alias usage is mixed across different language contexts. I hope this helps clarify: ![image](https://user-images.githubusercontent.com/6784/178674482-f6921cb0-2aa8-46f0-b404-da0a2ca0ce3e.png)

Apologies for such a long delay in response, but taking a look now. Unfortunately this isn't working for me locally. As example, I searched for `interactive` in the repo and...

Oh ... so this actually the opposite compatibility problem to the issues discussed in https://github.com/dgtized/winnow.el/pull/4. You are entirely correct though, I guess leaning so heavily on `keep-lines` is pretty fragile...

I *believe* this issue was resolved by #532 and #541. I know there is still work on the nrepl side to improve the new behavior around `cljr-slash`, and at some...

Cool, happy to review. I *think* the remaining issues on https://github.com/clojure-emacs/refactor-nrepl/issues/384 is probably more related to https://github.com/clojure-emacs/clj-refactor.el/issues/530 than this issue but happy to hold onto both if you see fit....

```elisp (defvar base (rspec-project-root (expand-file-name "spec/lib/gender_spec.rb"))) ;; Both spec/lib/gender_spec.rb & lib/gender.rb exist (file-relative-name (rspec-target-file-for (expand-file-name "spec/lib/gender_spec.rb")) base) "lib/gender.rb" (file-relative-name (rspec-spec-file-for (expand-file-name "lib/gender.rb")) base) "spec/gender_spec.rb" ;; When running C-c , t...

``` rspec-primary-source-dirs => ("app" "lib") ``` Are you suggesting I need to adjust that variable differently?

Awesome! I used: (setq rspec-primary-source-dirs '("app")) And it fixed it. Happy to close this given it now works, but wonder if should include this example in the documentation somewhere? Or...