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

Исправления русского, битых ссылок и новый перевод одного из блоков в container #185

Merged
merged 3 commits into from Jan 18, 2017

Conversation

s-belichenko
Copy link

  1. В примере такого слова не встречается, заметил на то, что подходит по смыслу (проверить, мог ошибиться).
  2. Ошибка в русском языке.

1. В примере такого слова не встречается, заметил на то, что подходит по смыслу (проверить, мог ошибиться).
2. Ошибка в русском языке.
@@ -258,7 +258,7 @@ Service Container (сервис-контейнер, ранее IoC-контей
<a name="contextual-binding"></a>
## Контекстное связывание

Иногда у вас может быть несколько реализаций одного интерфейса и вы хотите внедрять их каждый в свой класс. Например, когда
Иногда у вас может быть несколько реализаций одного интерфейса и вы хотите внедрять каждую из них в свой класс. Например, когда
Copy link
Collaborator

Choose a reason for hiding this comment

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

В оригинале

Sometimes you may have two classes that utilize the same interface, but you wish to inject different implementations into each class.

Может заменить на что-то более понятное и близкое к оригиналу?

Copy link
Author

Choose a reason for hiding this comment

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

Согласен, в английской версии более понятно читается, в русской предложение задом наперед повернули, хотя смысл-то тот же, конечно. У тебя есть права поменять мой коммит, или мне откатить и перевести?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@s-belichenko-sold ты можешь в этой же ветке новый коммит добавить и пул этот же обновить.

Stanislav added 2 commits January 18, 2017 02:22
1. Задвоилась версия в ссылке на container.
2. Перевел заголовок последнего подраздела.
Изменил как перевод, так и пример, сделав их как можно ближе к оригиналу, с учетом особенностей перевода.
@s-belichenko s-belichenko changed the title Ошибка в описании примера и небольшая орф. правка Исправления русского, битых ссылок и новый перевод одного из блоков в container Jan 18, 2017
@s-belichenko
Copy link
Author

s-belichenko commented Jan 18, 2017

@vladshcherbin Слушай, а когда реквест обновляешь тут, то старые комментарии затираются, что ли? - upd: увидел, где старые комментарии.
Ну и в общем отвечая на тот комментарий - я считаю, что лучше откат реквеста, ресет коммита и заново его зареквестить, в итоге меньше коммитов в основном форке.
Да, ну и перевод конкретно так переписал, посмотрел как пример сделан в оф. документации - земля и небо с нашим, пример в итоге стал понятнее. Но за мной проверить надо обязательно, я так-то пока не слишком настоящий сварщик.

@vladshcherbin vladshcherbin merged commit a451e9b into LaravelRUS:5.3 Jan 18, 2017
@vladshcherbin
Copy link
Collaborator

vladshcherbin commented Jan 18, 2017

@s-belichenko-sold нет, не затираются. Там выше есть container.md и справа есть кнопочка show outdated. Так можно читать комменты по коммитам.

screen shot 2017-01-18 at 3 18 43 am

Откат не лучше т.к. ты можешь у себя хоть 10 коммитов сделать, а принимающий твой pull может их все склеить в 1 коммит чтобы не засорять git ветку (твои 3 коммита превратились в 1).

Ну а вообще, лучше не в мастер конечно коммитить у себя на компе, а создать ветку, коммитить в неё и потом уже пулить её в основной репо. Таким образом, если у тебя несколько pull request-ов в основной репозиторий, ты легко можешь переключаться между ними и делать сколько угодно коммитов не пересекаясь.

Надеюсь, понятно написал. Если что, спрашивай 😉

@s-belichenko
Copy link
Author

s-belichenko commented Jan 18, 2017

Так можно читать комменты по коммитам

Ага, уже увидел, выше комментарий проапдейтил.

твои 3 коммита превратились в 1

О, я как-то и не проверял, а сам никогда реквесты не принимал - опять же, спасибо.

не в мастер конечно коммитить у себя на компе, а создать ветку

Да я, если честно, такие правки вообще через веб делаю :) Да, спасибо еще раз, все понятно.

@vladshcherbin
Copy link
Collaborator

vladshcherbin commented Jan 18, 2017

@s-belichenko-sold я тоже зачастую в веб версии такие правки делаю. В ней ты можешь делать так: выбираешь файл, редактируешь его, потом снизу выбираешь коммитить в новую ветку.

screen shot 2017-01-18 at 3 33 46 am

Эта ветка создаётся в твоём форке как пулл реквест. Если что-то нужно добавить - выбираешь эту ветку в своём форке, редактируешь что нужно и сохраняешь. Все изменения будут в этой ветке и, соответственно, в одном пул реквесте к твоему форку.

После того, как ты доволен пулом, ты переходишь в основной репозиторий и делаешь пул уже в него, выбирая ветку из своего форка (эта ветка - и есть твой пул реквест в свой форк).

Звучит вроде сложно, но на деле просто, надо только попробовать 😉

@s-belichenko
Copy link
Author

@vladshcherbin не-не, все просто, тут дело в том, что я пул-реквесты никогда со стороны владельца не видел. Спасибо еще раз!

@s-belichenko
Copy link
Author

@vladshcherbin да, я кстати не воспользовался этой инструкцией, а там написано про номер коммита, если делать перевод вроде как я сделал блока про контекстное связывание.
http://laravel.su/articles/rus-documentation-contribution-guide

@vladshcherbin
Copy link
Collaborator

@s-belichenko-sold ничего страшного, т.к. ты просто обновил пример.

Текущий же файл в английской версии сильно отличается, да и пример там уже другой. Номер коммита обновляется если файл полностью совпадает с соответствующим файлом такого же коммита из английской версии.

Различия screen shot 2017-01-18 at 4 29 50 am

@s-belichenko
Copy link
Author

Согласен, область, на которых там даются примеры, совершенно другая. Такое произошло, потому что в русской версии были примеры изначально свои кем-то задуманы, или просто Тейлор когда-то область примеров решил поменять?

Про отличия английской версии не понял, в смысле, что у нее отличия там между своими собственными версиями давно уже серьезные, или и это, и то, что в русской версии тоже уже давно своё все?

@s-belichenko
Copy link
Author

Впрочем, кажется понял: чуть раньше удивился, почему ты при моем коммите в container посмотрел оригинальную документацию, приведя цитату оттуда, а сейчас начал сравнивать документацию тут и там и вижу, что отличий действительно масса, при этом критичных. Неожиданно :)

@vladshcherbin
Copy link
Collaborator

vladshcherbin commented Jan 18, 2017

@s-belichenko-sold ну смотри, в файле номер 4deba2bfca6636d5cdcede3f2068eff3b59c15ce

Список версий файла в английской версии. Сейчас русская версия сокращённо 4deba2b. Вот файл на момент этой версии. Таким образом, наша русская версия совпадает с этой английской версией. А вот актуальная последняя версия этого файла.

Получается, что с момента перевода русской версии Тейлор несколько раз обновил английскую версию и наша русская устарела.

Соответственно, если кто-то полностью обновит русскую версию, чтобы она совпадала с текущей английской, тогда и нужно будет поменять номер на номер английского коммита.

Я смотрел именно старую версию, это предложение совпадает. А вот дальше уже нет 🙂

старая

Sometimes you may have two classes that utilize the same interface, but you wish to inject different implementations into each class. For example, when our system receives a new Order, we may want to send an event via PubNub rather than Pusher.

новая

Sometimes you may have two classes that utilize the same interface, but you wish to inject different implementations into each class. For example, two controllers may depend on different implementations of the Illuminate\Contracts\Filesystem\Filesystem contract.

@s-belichenko
Copy link
Author

s-belichenko commented Jan 18, 2017

Ага, в общем лучше все оставить, как есть :)

Про номер коммита когда увидел в инструкции, решил что все не так банально (типа есть некий инструмент, позволяющий что-то более интеллектуальное делать), но это видимо спать уже надо идти.

Про версии файлов я когда переводил еще не видел той инструкции и соответственно не знал как посмотреть актуальную версию английской документации относительно нашей версии, теперь понятно. Я-то тупо в адресе к доке поменял su на com и посмотрел ту последнюю версию, которая выложена у Тейлора на сайте.

@vladshcherbin
Copy link
Collaborator

@s-belichenko-sold там в инструкции написано, что можно через программку смотеть diff английских версий и команды в списке перевода (например git difftool b5117ea f5229b5 artisan.md), может так удобней. Мне просто привычно через гитхаб :)

@s-belichenko
Copy link
Author

@vladshcherbin да не, согласен что через web удобнее, проще уж тогда в PHPStorm склонировать репу и в нем и смотреть.

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.

None yet

2 participants