Skip to content

Conversation

@ashton314
Copy link
Contributor

Emacs 31 makes if-let and when-let macros obsolete. I'm getting warning messages whenever I boot up Emacs about this. This commit switches if-letif-let* and when-letwhen-let*. The warning goes away with this change.

Also, fixed a typo in CONTRIBUTING.org.

@bdarcus
Copy link
Contributor

bdarcus commented Oct 24, 2025

Thanks, though the CI is complaining, in one case because we're testing an old version of Emacs! Not sure why the other error ATM.

While were here, is it worth bumping the version number, or minimum required Emacs version?

@roshanshariff
Copy link
Collaborator

I can't check right now, but does depending on the compat package help? I would hope it defines those starred macros for older Emacs versions as well.

@bdarcus
Copy link
Contributor

bdarcus commented Oct 24, 2025

I can't check right now, but does depending on the compat package help? I would hope it defines those starred macros for older Emacs versions as well.

It looks like yes:

https://elpa.gnu.org/packages/doc/compat.html

Can you give that a try @ashton314?

@ashton314
Copy link
Contributor Author

I'm not familiar with compat, but I'll give it a shot.

@ashton314
Copy link
Contributor Author

@bdarcus Added compat. I'm not sure what version to use—I'm just following what minad has in his vertico package.

@bdarcus bdarcus merged commit 866f85a into emacs-citar:main Oct 27, 2025
3 of 6 checks passed
@bdarcus
Copy link
Contributor

bdarcus commented Oct 27, 2025

Thanks @ashton314 !

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