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

Update editors info #225

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sogaiu
Copy link
Contributor

@sogaiu sogaiu commented Jun 26, 2024

This PR is an attempt to address #222.

The initial content consists of:

  • Creation of a separate editor section (broken out from being part of the "Modules and Libraries" section)
  • An introductory paragraph explaining the content along with mentioning the "editors and tooling" discussion area in the Zulip Instance
  • Changing the list of currently supported editors
    • Atom has been removed because it was sunset (the link wasn't leading anywhere useful AFAICT)
    • Helix and TIC-80 were added
    • The original ordering has been mostly preserved with new things being appended to the end of the list
  • Each editor link [1] points to a page at this repository. Each such page there describes some supported features along with some relevant links.

This last change (pointing to a separate repository with descriptions) has the potential benefit of not having to update the list at janet-lang.org so frequently.

Don't know if folks are comfortable with this last point, but this is my initial proposal (^^;


[1] To make it easy to see those links, here they are:

Copy link
Member

@pepe pepe left a comment

Choose a reason for hiding this comment

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

LGTM.

@sogaiu sogaiu marked this pull request as ready for review June 29, 2024 00:43
@sogaiu
Copy link
Contributor Author

sogaiu commented Jun 29, 2024

I've removed the draft status of this PR.

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