Skip to content

add claude-code.el#560

Merged
jcs090218 merged 4 commits into
emacs-tw:masterfrom
tninja:master
Jun 28, 2025
Merged

add claude-code.el#560
jcs090218 merged 4 commits into
emacs-tw:masterfrom
tninja:master

Conversation

@tninja

@tninja tninja commented Jun 21, 2025

Copy link
Copy Markdown
Contributor

and sort packages by alphabet order.

@jcs090218 PTAL Thanks

@jcs090218

Copy link
Copy Markdown
Member

Can you revert the order due to https://github.com/emacs-tw/awesome-emacs?tab=readme-ov-file#the-order-of-items.

Please don’t rearrange the package ordering without any reason! The items should be sorted by its popularity ( roughly, because it’s impossible to have a precise standard. e.g. most people would use it; recommended for every newbie…), instead of your personal preference.

Otherwise, LGTM! ;)

@tninja

tninja commented Jun 22, 2025

Copy link
Copy Markdown
Contributor Author

Got it. The order has been reverted.

@tninja

tninja commented Jun 22, 2025

Copy link
Copy Markdown
Contributor Author

CI fail is not relevant to this PR:

Screenshot from 2025-06-22 06-44-51

Would you mind merge the PR? I cannot do that. @jcs090218 Thanks

@jcs090218

Copy link
Copy Markdown
Member

I don't think Other AI tools is a good category since it’s quite confusing. 🤔

@tninja

tninja commented Jun 22, 2025 via email

Copy link
Copy Markdown
Contributor Author

@tninja

tninja commented Jun 22, 2025 via email

Copy link
Copy Markdown
Contributor Author

@tninja

tninja commented Jun 26, 2025

Copy link
Copy Markdown
Contributor Author

Ping this thread @jcs090218 What is the next step before merge? Let me know if we want to

  1. Update the Other AI tools => Some other title
  2. Split this into two PRs.

@jcs090218

Copy link
Copy Markdown
Member

I recommend split into 2 PRs so I can merge claude-code.el. 🤔 Then we can further discuss about the mcp section.

@tninja

tninja commented Jun 26, 2025

Copy link
Copy Markdown
Contributor Author

reverted

@tninja

tninja commented Jun 28, 2025

Copy link
Copy Markdown
Contributor Author

@jcs090218 can you merge this PR? MCP one is splitted now: #561

@jcs090218 jcs090218 merged commit 6c29284 into emacs-tw:master Jun 28, 2025
2 of 3 checks passed
@jcs090218

Copy link
Copy Markdown
Member

Thank you!

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.

2 participants