Ёфикация

[lex111]

По факту мы уже используем её, но не везде. Давайте исправим этот досадное недоразумение.

Использовал https://github.com/e2yo/eyo плюс собственные правки, наверное что-то упустил, но было бы неплохо, если кто-нибудь посмотрит, всё ли в порядке. Понимаю, что такое количество изменённых строк нереально посмотреть, но хотя бы выборочно. @saundefined поможешь в этом?

[saundefined]

@lex111, да, конечно, пробегусь тоже

[saundefined]

@lex111 почитал выборочно файлы — ошибок не нашёл.
Предлагаю добавить линтер в GH-actions

name: Check Russian build

on: [ push, pull_request ]

jobs:
  check-build:
    name: Check build
    runs-on: ubuntu-latest
    steps:
      - name: Fetch Russian module
        uses: actions/checkout@master
        with:
          path: 'ru'

      - name: Fetch English module
        uses: actions/checkout@master
        with:
          repository: 'php/doc-en'
          path: 'en'

      - name: Fetch base module
        uses: actions/checkout@master
        with:
          repository: 'php/doc-base'
          path: 'doc-base'

      - name: Check build
        run: php doc-base/configure.php --with-lang=ru

  e2yo-validation:
    name: Check build
    runs-on: ubuntu-latest
    needs:
      - check-build

    strategy:
      matrix:
        node-version: [ 10.x ]

    steps:
      - uses: actions/checkout@v2
      - name: Use Node.js ${{ matrix.node-version }}
        uses: actions/setup-node@v1
        with:
          node-version: ${{ matrix.node-version }}
      - run: npm install eyo -g
      - run: eyo --lint "**/*.xml"

[lex111]

@saundefined отлично, спасибо, по поводу линтера: я думаю ещё поисправлять опечатки при помощи https://github.com/hcodes/yaspell, который поддерживает проверку буквы "ё" https://github.com/hcodes/yaspeller/blob/master/README.ru.md#--check-yo. Так что в следующем PR я добавлю такую проверку в CI вместе с исправлением опечаток (там очень их много).

[lex111]

Кстати, а не лучше было бы упросить текущий CI-файл, как это сделано в китайском переводе? https://github.com/php/doc-zh/blob/master/.github/workflows/build.yml#L24

[saundefined]

Спасибо, упростил.

Ещё, может, translation_standards.xml в вики перенести?

[lex111]

@saundefined да, наверное стоит, кстати у меня через curl http://url_of_github_patch.patch | git am не удалось применить патч, это точно рабочая команда? Поэтому воспользовался wget, как описано в японском переводе https://github.com/php/doc-ja#%E3%83%A1%E3%83%B3%E3%83%86%E3%83%8A%E5%90%91%E3%81%91%E3%81%AE%E6%B3%A8%E6%84%8F

Вот что я получил, выполнив curl https://github.com/php/doc-ru/pull/3.patch | git am :

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   134  100   134    0     0    300      0 --:--:-- --:--:-- --:--:--   300
Patch is empty.
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Я что-то не так сделал?

[nikic]

@lex111 curl -L https://github.com/php/doc-ru/pull/3.patch | git am should work

[lex111]

@nikic cool, thanks!

BTW can you please enable issues for the docs repos? This can be useful, for example, if someone is not ready to fix a typo by opening PR, but wants to report it.

[nikic]

@lex111 I've enabled issues for just this repo. Normally we use the issue tracker at bugs.php.net, but I think for translations using github issues probably makes more sense. Let's see how it goes...