Fuco1 / smartparens

Minor mode for Emacs that deals with parens pairs and tries to be smart about it.
GNU General Public License v3.0
1.8k stars 193 forks source link

Ruby `def` no longer completes `end` for me #1176

Closed vemv closed 3 months ago

vemv commented 7 months ago

Using ruby-mode, typing def SPC no longer inserts end for me.

I'm at a loss as to why this happened. I tried going back N months back per Git history.

My smartparens setup is vanilla/minimalistic; I've been a user for some 5 years (as I've hacked on Emacs for 10+).

What are some measures to debug this? Can I trigger the completion by hand? Or otherwise inspect the ruby- / smartparens-related environment?

Maybe there's some sort of cross-package interference?

Steps to reproduce the problem

May be provided on request

Environment & version information


Thanks - V

vemv commented 7 months ago

...I use Emacs 27 (as it's good enough, and some packages I use break on 28). I see that the CI has:

          - '25.1'
          - '25.2'
          - '25.3'
          - '26.1'
          - '26.2'
          - '26.3'
          - 'snapshot'

which surely means that 27 is getting skipped.


Update: I went back to 26, made no difference.

grimnebulin commented 6 months ago

I also started seeing this recently. I tried the following steps:

No end was inserted.

Not only that, but sexp-based motion commands have stopped working properly, probably at the same time. For example, when point is after the end at the end of a function and I invoke sp-backward-sexp, point used to move to start of the corresponding def. Now it just moves to the start of the end. sp-backward-up-sexp does nothing at all anymore, as far as I can tell.

vemv commented 6 months ago

FWIW, I never got to diagnose this and am using https://github.com/rejeep/ruby-end.el for now

grimnebulin commented 5 months ago

I updated to from Emacs 27 to Emacs 29 and this functionality now works as I'm used to. Emacs 28 didn't cut it.

Fuco1 commented 3 months ago

It probably got broken in this commit 4e3ca735aadc570a5b4b456519000d6048785bf4 . I reverted some changes and made the config load in the legacy mode as well.

vemv commented 3 months ago

Thanks much!

Looking forward to revert ruby-end.el out of my config :) it worked but obviously it's far less capable.