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

First pass at updating en/documentation/index.md #3427

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zenspider
Copy link

@zenspider zenspider commented Nov 20, 2024

main changes:

LOTS of bitrot on this page. Most has been removed but some have been
moved down to an "older" section.

Revamped editor section to include difficulty "levels"

Added a bunch of sections and links mostly derived from the ruby
discord's #library.

This fixes #3426

@zenspider zenspider requested a review from a team as a code owner November 20, 2024 05:43
LOTS of bitrot on this page. Most has been removed but some have been
moved down to an "older" section.
@kaya3
Copy link

kaya3 commented Nov 21, 2024

From my (new to Ruby) perspective, this looks like a massive improvement.

Having just completed the Ruby Koans, I can confirm that they are still correct for Ruby 3.3.

Copy link
Member

@st0012 st0012 left a comment

Choose a reason for hiding this comment

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

This is a HUGE improvement ❤️
Just a few suggestions 🙂

[The Well-Grounded Rubyist][grounded]
: A tutorial that begins with your first Ruby program and takes you all the way to sophisticated topics like reflection, threading, and recursion.

#### Adept
Copy link
Member

Choose a reason for hiding this comment

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

As a non-native English speaker, I think Adept is a relatively rare choice of word. Maybe Advanced will be more approachable?

Choose a reason for hiding this comment

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

Maybe Experienced would be better? Advanced and Expert are closer in my mind than Adept/Experienced and Expert are.

Copy link
Author

Choose a reason for hiding this comment

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

Intermediate?

* Expert
* [Emacs][20] with [Ruby mode][21] or [Enhanced Ruby mode][enh-ruby-mode]
* [Vim][25] with [vim-ruby][26] plugin
* [NeoVim][neovim]
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure if we need or should pair editors with experience levels. For example, I started writing Ruby with Vim, then switched to NeoVim for like 6 years, and switched to VS Code like 2 years ago.

Choose a reason for hiding this comment

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

I think the idea is from an approachability standpoint instead of an end user skill level standpoint.

Copy link
Author

Choose a reason for hiding this comment

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

yeah, I meant this more about learning curve of the editor

Comment on lines 158 to 159
All of these editors support the Language Server Protocol (LSP). You
can use Shopify's [ruby-lsp][ruby-lsp] plugin for ruby support.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
All of these editors support the Language Server Protocol (LSP). You
can use Shopify's [ruby-lsp][ruby-lsp] plugin for ruby support.
All of these editors support the Language Server Protocol (LSP), either by default or through their LSP plugins.
Shopify's [ruby-lsp][ruby-lsp] is one of the most popular language servers for Ruby and [supports all of the above editors](https://shopify.github.io/ruby-lsp/editors.html)

@zenspider
Copy link
Author

Pushed some edits based on feedback.

Please let me squash and reword before merge (assuming).

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.

Outdated "getting started" links
4 participants