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

Sync clojure-mode-extra-font-locking.el version with clojure-mode.el #679

Closed
wants to merge 0 commits into from

Conversation

manphiz
Copy link
Contributor

@manphiz manphiz commented Mar 25, 2024

  • It looks like the version of clojure-mode-extra-font-locking.el has been out-of-sync with clojure-mode.el and has been stuck in 3.0.0 for a long time.
  • This patch sync the version with the current snapshot.

Note: it would be great to fix 5.18.x branch but not really needed I guess.

  • The commits are consistent with our [contribution guidelines][1].
  • You've added tests (if possible) to cover your change(s). Bugfix, indentation, and font-lock tests are extremely important!
  • You've run M-x checkdoc and fixed any warnings in the code you've written.
  • You've updated the changelog (if adding/changing user-visible functionality).
  • You've updated the readme (if adding/changing user-visible functionality).

@bbatsov
Copy link
Member

bbatsov commented Mar 25, 2024

That's intentional. It's a separate package, so it should have a different version number.

@manphiz
Copy link
Contributor Author

manphiz commented Mar 25, 2024

I see. However, it looks like on stable.melpa.org the version of clojure-mode-extra-font-locking is the same as clojure-mode[1]. This may cause an issue that a locally built clojure-mode-extra-font-locking may get superseded by the version on stable.melpa.org, but they are actually the same version. Is there a way to let stable.melpa.org honor the version specified in the source file?

[1] https://stable.melpa.org/#/clojure-mode-extra-font-locking

@bbatsov
Copy link
Member

bbatsov commented Mar 26, 2024

I don't think so, as MELPA uses the newest tag as the version instead of the version metadata. Probably the only solution would be to move clojure-mode-extra-font-locking to a separate repo. CIDER's dynamic font-locking made this package somewhat obsolete, so I haven't thought much about it recently.

@manphiz
Copy link
Contributor Author

manphiz commented Mar 26, 2024

Ack. Please feel free to close this (or leave open as you like :)

@jackrusher
Copy link

Additional semi-relevant question: did we lose font-locking in the mini-buffer with these changes in version 3.0?

@bbatsov
Copy link
Member

bbatsov commented Sep 24, 2024

@jackrusher For CIDER's eval commands? I don't recall if we ever had font-locking there.

@jackrusher
Copy link

@bbatsov I must have been hallucinating, but I swear there was syntax highlighting on results in the mini-buffer up until cider 3.0.0 🤔

@bbatsov
Copy link
Member

bbatsov commented Sep 24, 2024

Digging in the history of CIDER I see this was the case about 10 years ago, but I'm not sure when and why we changed this. Definitely not related to any changes in clojure-mode, though.

@bbatsov
Copy link
Member

bbatsov commented Sep 24, 2024

@jackrusher After digging a bit more - seems that's a recent regression caused by clojure-emacs/cider#3661

@bbatsov
Copy link
Member

bbatsov commented Sep 24, 2024

And it's now fixed in clojure-emacs/cider#3743 Turned out to be a trivial issue of using the wrong local variable.

@jackrusher
Copy link

@bbatsov great, thanks!

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.

3 participants