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

Removed unnecessary words from Getting Started docs #37946

Merged
merged 1 commit into from Mar 23, 2020

Conversation

sarcas
Copy link
Contributor

@sarcas sarcas commented Dec 12, 2019

[ci skip]

Summary

This PR removes some uses of "simple" and "just" from the getting started documentation.

Other Information

Using "simple" and "just" in documentation is very tempting! It is there to show enthusiasm and to encourage new people to try out the thing the author loves. In this case, Rails. However, if you are following documentation and struggle with it, reading these words can be a source of frustration.

Also: using fewer words can make the instructions more clear and can reduce ambiguity.

Using "simple" and "just" in documentation is very tempting! It is there
to show enthusiasm and to encourage new people to try out the thing the
author loves. In this case, Rails. However, if you are following
documentation and struggle with it, reading these words can be a source
of frustration.

Also: using fewer words can make the instructions more clear and can
reduce ambiguity.
@rails-bot rails-bot bot added the docs label Dec 12, 2019
Copy link
Contributor

@kaspth kaspth left a comment

Choose a reason for hiding this comment

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

Sweet! I was curious about clarification on some of these

guides/source/getting_started.md Show resolved Hide resolved
guides/source/getting_started.md Show resolved Hide resolved
guides/source/getting_started.md Show resolved Hide resolved
guides/source/getting_started.md Show resolved Hide resolved
@rails-bot
Copy link

rails-bot bot commented Mar 22, 2020

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
Thank you for your contributions.

@rails-bot rails-bot bot added the stale label Mar 22, 2020
@@ -525,7 +525,6 @@ method called `form_with`. To use this method, add this code into
```

If you refresh the page now, you'll see the exact same form from our example above.
Building forms in Rails is really just that easy!
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't it make more sense to keep this as "Building forms in Rails is really easy!" I've seen how other technologies are used to build forms and I can guarantee you that it's definitely a lot easier to build forms in Rails than in any other framework out there. We could even say that "Building forms in Rails is fast and straightforward".

Copy link
Contributor

Choose a reason for hiding this comment

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

It's not easy if you have never done any programming before and the audience of a Getting Started page is more likely to be new programmers than seasoned switchers from another framework. ref: https://justsimply.dev

Choose a reason for hiding this comment

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

Maybe it would be more precise to say that it needs less typing. A lot of the Magic of Rails™️ amounts to not having to type 500 characters where 50 would do.

@rails-bot rails-bot bot removed the stale label Mar 22, 2020
@kaspth kaspth merged commit 520d774 into rails:master Mar 23, 2020
@kaspth
Copy link
Contributor

kaspth commented Mar 23, 2020

Thanks, for the code and reviews people! Ended up looking at this again and didn't think it was worth the extra rounds to shuffle more things around. It's overall better, so let's use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants