-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Removed more belittling words and expressions #11568
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
Conversation
@@ -38,8 +38,7 @@ you can use as is or as a start for your very own. It will start with a simple | |||
framework and more features will be added with time. Eventually, you will have | |||
a fully-featured full-stack web framework. | |||
|
|||
And of course, each step will be the occasion to learn more about some of the | |||
Symfony Components. | |||
Each step will be the occasion to learn more about some of the Symfony Components. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are using "Symfony components" (2/3) and "Symfony Components" (1/3) throughout the documentation.
Which one should we use?
I would propose a PR after this one is merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome 🎉
I like the removals of the usages of 'obvious' and 'of course'. I'm not so sure about removing 'easily' everywhere. In some cases it is there to tell the user that something will in face be easier then the 'normal' way or the native php way. For example right here: https://github.com/symfony/symfony-docs/pull/11568/files#diff-3e14f8d0c6a18bdad98baec449d3285dR8 From what i can see most of the easily can indeed be removed, but there are a few, like the one i just mentioned where it may be kept. Or maybe another word can be used that is more fitting. What do you think? |
@BackEndTea I've reviewed again all removals of "easily" and I still think we should remove all of them. For the "Form" component specially, I would never associate "easy" / "easily" with it (I would associate to it other positive words such as "powerful", "comprehensive", etc.). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great! i thought i had caught all of that in the pull request i did a while ago... though i think i only did it against what was the master branch then.
No description provided.