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

Fix all clojure ts related major modes #288

Merged
merged 1 commit into from
Mar 31, 2024

Conversation

dalugm
Copy link
Contributor

@dalugm dalugm commented Mar 4, 2024

After this pr

clojure-emacs/clojure-ts-mode#36

The names of clojure-ts's major modes had changed

@raxod502
Copy link
Member

We should keep the old names for backwards compatibility, then, right?

After clojure-ts-mode's this pr

clojure-emacs/clojure-ts-mode#36

The names of clojure-ts's major modes had changed

---

no changelog update needed
@dalugm
Copy link
Contributor Author

dalugm commented Mar 23, 2024

We should keep the old names for backwards compatibility, then, right?

I don't think so... In fact I highly doubt does anyone actually use it to program before v0.2.1. Because before v0.2.1, clojure-ts-mode can only highlight syntax, it lacked many features in clojure-mode and it could not work with cider (I believe 90% clojurists in Emacs use it). Even now (v0.2.2, the name changed happen in v0.2.1), it's just worked and still need improve.

But... I found some major modes like clojure-ts-clojuredart-mode are drived from clojure-ts-mode. We can keep the base mode only.

Copy link
Member

@raxod502 raxod502 left a comment

Choose a reason for hiding this comment

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

✨ Ok!

@raxod502 raxod502 merged commit c7040a6 into radian-software:main Mar 31, 2024
4 checks passed
@dalugm dalugm deleted the patch-1 branch March 31, 2024 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants