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

New package: Lean #50

Merged
merged 2 commits into from
May 5, 2024
Merged

New package: Lean #50

merged 2 commits into from
May 5, 2024

Conversation

gebner
Copy link
Contributor

@gebner gebner commented Oct 12, 2022

This PR adds a package for the Unicode abbreviations used by the editor plugins for the Lean theorem prover.

@federico-terzi
Copy link
Contributor

Hey @gebner ,

Looks great! Thank you for the help :) Could you please pull from the latest main and repush here? Otherwise the CI won't be able to run correctly

Thanks! :)

@smeech smeech self-assigned this May 2, 2024
@smeech
Copy link
Collaborator

smeech commented May 2, 2024

Hey @gebner ,

Looks great! Thank you for the help :) Could you please pull from the latest main and repush here? Otherwise the CI won't be able to run correctly

Thanks! :)

No response to @federico-terzi's request 2½ years ago.
This is one of two Lean PRs, the other being the slightly broken #107.
No cmd, script or shell on search, but over 4,000 expansions, mainly with hex-code representation of obscure characters.
Interesting reversal of the conventional - trigger:/replace: pairs - I was interested to see this still works.

@AucaCoyan
Copy link
Member

I gave a quick glance over a few lines and it seems alright.
I hope I don't break anything by quoting the infamous LGTM (looks good to me). We can revert if any issues are raised 👍🏼 .
Thank you Smeech and @gebner !

@AucaCoyan AucaCoyan merged commit 53d1dcd into espanso:main May 5, 2024
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.

4 participants