Skip to content

Emacs 30 1 stewarding #37

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

Merged
merged 8 commits into from
Mar 27, 2025
Merged

Emacs 30 1 stewarding #37

merged 8 commits into from
Mar 27, 2025

Conversation

xvw
Copy link
Member

@xvw xvw commented Mar 27, 2025

Some fixes after 30.1 (fix #31)

@xvw xvw requested review from tmcgilchrist and voodoos March 27, 2025 01:55
@xvw xvw force-pushed the emacs-30-1-stewarding branch 4 times, most recently from b1ec68a to 1e5ebd2 Compare March 27, 2025 02:25
@xvw xvw force-pushed the emacs-30-1-stewarding branch from 1e5ebd2 to 0e55188 Compare March 27, 2025 02:38
configuration in this way:

```diff
(use-package ocaml-eglot
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I"m guessing you can also add a use-package invocation for flycheck-eglot, so this example would be more complete.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed on 74f0699

+ :config
+ (add-hook #'ocaml-eglot-hook
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's slightly different from your example above. (in the auto-format section) Perhaps you want to make them consistent?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you're looking at the source and not the modification. In the diff, I relay to :hook ... Does this work for you?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I didn't see this was a "diff". My bad!

Btw, I saw this block using "Scheme" as the language, but GH actually supports elisp and emacs-lisp for fenced code blocks:

(use-package ocaml-eglot
  :ensure t
  :after tuareg
  :hook
  (tuareg-mode . ocaml-eglot)
  (ocaml-eglot . eglot-ensure))

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes! Fixed!

@@ -294,14 +302,14 @@ If there is no available holes, it returns the first one of HOLES."

(defun ocaml-eglot--first-hole-in (start end)
"Jump to the first hole in a given range denoted by START and END."
(when-let ((hole (ocaml-eglot--get-first-hole-in start end))
(when-let* ((hole (ocaml-eglot--get-first-hole-in start end))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The line below is not properly indented.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I run indent-buffer on 7f8ab06

Copy link
Contributor

@bbatsov bbatsov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall the changes seem fine to me.

@xvw
Copy link
Member Author

xvw commented Mar 27, 2025

Thanks a lot for your review @bbatsov!
I apply your feedbacks!

@xvw xvw force-pushed the emacs-30-1-stewarding branch from 74f0699 to 69fd31b Compare March 27, 2025 12:37
@xvw xvw merged commit b2553de into main Mar 27, 2025
5 checks passed
@xvw xvw deleted the emacs-30-1-stewarding branch March 27, 2025 13:30
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.

Emacs 30.x follow-up Document integration with flycheck
2 participants