* Fix error on Pull Request creation
Fix https://github.com/github/docs-engineering/issues/1293. This
solution does not really handle the error, it just prevents the script
from stopping, since sometimes we get a timeout error from the load
balancer even though the request was successfully handled by rails.
* Add a nudge to the localization folks when a batch is created
Ever since the translation batches are created and merged automatically
it is difficult to see when they started or finished correctly. Also,
sometimes, docubot automatically closes some pull requests, and at the
time of this writing I don't know the reason.
This change intends to make batches not go unnoticed. We can revert it
later if this becomes too noisy and the process is stable.
* fix: remove ping to docs-localization team from batch PR body
Discussion: https://github.com/github/docs-internal/pull/23633/files#r771896816
* refactor: move report-reset-files into its own step
* refactor: move git push into its own step
* spike: move PR creation into its own script
* feat: annotate createTranslationBatchPullRequest fn
* enhancement: embed nwo#pr format in logger
* fix: import fs directly
* fix: import @actions/github directly
* docs: update annotations in fn block
* build: update step name to be self-documenting
* Retry on 502
* Rename method to reflect its behaviour
* Update codeowners
* Fix comment
* Update .github/actions-scripts/create-translation-batch-pr.js
Co-authored-by: Peter Bengtsson <peterbe@github.com>
* Update .github/actions-scripts/create-translation-batch-pr.js
Co-authored-by: Peter Bengtsson <peterbe@github.com>
* Update .github/actions-scripts/create-translation-batch-pr.js
Co-authored-by: Peter Bengtsson <peterbe@github.com>
* Appease linter and fix script
* Move options to main
* Rename var
* Refactor options
* Check for response
* First try to find and then to create the PR
* Update .github/actions-scripts/create-translation-batch-pr.js
Co-authored-by: Peter Bengtsson <peterbe@github.com>
* Update .github/actions-scripts/create-translation-batch-pr.js
Co-authored-by: Peter Bengtsson <peterbe@github.com>
* Update .github/actions-scripts/create-translation-batch-pr.js
Co-authored-by: Francis <15894826+francisfuzz@users.noreply.github.com>
* Remove unreachable conditional
* Lint
* fix: remove duplicate annotation
Co-authored-by: docubot <67483024+docubot@users.noreply.github.com>
Co-authored-by: Francis <15894826+francisfuzz@users.noreply.github.com>
Co-authored-by: Peter Bengtsson <peterbe@github.com>
* Add permission matrices to all Actions workflows
Also cleanup a few token references
* Add actions:read permissions for CodeQL
* Add prs:read permissions for unit test workflow
* Enable auto-merge for translation-batches
* Attempt to approve and set auto-merge
* Temporarily disable check for tests
* Test commit. Revert later.
* Not self approving anymore
* Improve failure message
* Fix auto-merge
* Remove test code
* Add ownership to @github-actions
* Delete obsolete files from Crowdin in workflow
* Remove empty files before `crowdin upload`
Those files would have been rejected by Crowdin.
Close https://github.com/github/crowdin-support/issues/117
* Change delete command
* Add crowdin translations
* Run script/i18n/homogenize-frontmatter.js
* Run script/i18n/fix-translation-errors.js
* Run script/i18n/lint-translation-files.js --check parsing
* Run script/i18n/lint-translation-files.js --check rendering
* run script/i18n/reset-files-with-broken-liquid-tags.js --language=pt
* run script/i18n/reset-known-broken-translation-files.js
* Check in pt CSV report
Co-authored-by: Marcelo Jacobus <marcelo.jacobus@gmail.com>
The new order will use "reset known broken files" as a last step. That
way we can better understand if the previous steps were enough to catch
those errors. The least files are reset in that last step, the better.
Also, the check for broken/mismatched liquid tags is performed after
rendering and parsing check. That helps us to identify how useful the
mismatch check is, if at all.
* Add script to report reset files
* Has the workflow report the reset files
* Fix script name
* Filter report by language
* Update .github/workflows/create-translation-batch-pr.yml
* Use build in array methods
