Skip to content

feat: mark invalid entries as fuzzy | FC-0012 #7

feat: mark invalid entries as fuzzy | FC-0012

feat: mark invalid entries as fuzzy | FC-0012 #7

Workflow file for this run

# Run pull request tests and validation for openedx-translations
name: CI
on:
- pull_request
jobs:
# Run unit and integration tests for Python
python-tests:
runs-on: ubuntu-latest
steps:
# Clones the openedx-translations repo
- name: clone openedx/openedx-translations
uses: actions/checkout@v3
- name: Install gettext
run: |
sudo apt install -y gettext
- name: setup python
uses: actions/setup-python@v4
with:
python-version: '3.8'
- name: Run python tests
run: |
make test_requirements
make test
# Validate translation files
validate-po-files:
needs: python-tests
runs-on: ubuntu-latest
steps:
- name: clone openedx/openedx-translations
uses: actions/checkout@v3
- name: Install gettext
run: |
sudo apt install -y gettext
- name: Allow editing translation files for bot pull requests
env:
# secrets can't be used in job conditionals, so we set them to env here
TRANSIFEX_APP_ACTOR_NAME: "${{ secrets.TRANSIFEX_APP_ACTOR_NAME }}"
TRANSIFEX_APP_ACTOR_ID: "${{ secrets.TRANSIFEX_APP_ACTOR_ID }}"
if: "${{ github.actor == env.TRANSIFEX_APP_ACTOR_NAME && github.actor_id == env.TRANSIFEX_APP_ACTOR_ID }}"
run: |
echo "VALIDATION_OPTIONS=--mark-fuzzy" >> "$GITHUB_ENV"
- name: Validate translation files
id: validate_translation_files
run: |
has_errors=0
python scripts/validate_translation_files.py $VALIDATION_OPTIONS 2>error_log.txt || has_errors=1
cat error_log.txt # Print the errors to the console for debugging
# Save the validation errors to an output variable
{
echo 'ERROR_LOG<<EOF'
fold -w 100 -s error_log.txt
echo EOF
} >> "$GITHUB_OUTPUT"
exit $has_errors
- name: Commit changes to git
if: ${{ always() && !github.event.pull_request.head.repo.fork }}
run: |
# Commit if there are changes to translation files
if ! git diff --no-ext-diff --quiet --exit-code; then
git config --global user.email "translations-bot@openedx.org"
git config --global user.name "edx-transifex-bot"
git add .
git commit -m "fix: mark invalid entries as fuzzy" -- translations/
fi
- name: Post translation validation results as a pull request comment
# Due to GitHub Actions security reasons posting a comment isn't possible on fork pull requests.
# This shouldn't be an issue, because bots writes directly to this repository.
if: ${{ always() && !github.event.pull_request.head.repo.fork }}
uses: mshick/add-pr-comment@7c0890544fb33b0bdd2e59467fbacb62e028a096
with:
message: |
:white_check_mark: All translation files are valid.
```
${{ steps.validate_translation_files.outputs.ERROR_LOG }}
```
This comment has been posted by the `validate-translation-files.yml` GitHub Actions workflow.
message-failure: |
:warning: There are errors in the translation files:
```
${{ steps.validate_translation_files.outputs.ERROR_LOG }}
```
This comment has been posted by the `validate-translation-files.yml` GitHub Actions workflow.
# Run commitlint on the commit messages in a pull request.
commitlint:
needs: validate-po-files
uses: openedx/.github/.github/workflows/commitlint.yml@master
# Automatically merge pull requests created by the "Transifex Integration" github app
# https://github.com/apps/transifex-integration
automerge-transifex-app-pr:
needs: commitlint
runs-on: ubuntu-latest
# Merges the pull request
steps:
- name: clone repository
uses: actions/checkout@v3
- name: auto-merge pull request
env:
# secrets can't be used in job conditionals, so we set them to env here
TRANSIFEX_APP_ACTOR_NAME: "${{ secrets.TRANSIFEX_APP_ACTOR_NAME }}"
TRANSIFEX_APP_ACTOR_ID: "${{ secrets.TRANSIFEX_APP_ACTOR_ID }}"
# This token requires Write access to the openedx-translations repo
GITHUB_TOKEN: ${{ secrets.EDX_TRANSIFEX_BOT_GITHUB_TOKEN }}
PR_NUMBER: ${{ github.event.number }}
if: "${{ github.actor == env.TRANSIFEX_APP_ACTOR_NAME && github.actor_id == env.TRANSIFEX_APP_ACTOR_ID }}"
run: |
# Add the pull request to the merge queue with --rebase commit strategy
gh pr merge ${{ github.head_ref }} --rebase --auto