Improve captions appearance when there is also a CTA #13546
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: Lint I18N | |
on: | |
push: | |
paths: | |
- '**.js' | |
- '**.cjs' | |
- '**.ts' | |
- '**.tsx' | |
- '**/package.json' | |
- 'package-lock.json' | |
- 'web-stories.php' | |
- 'includes/**.php' | |
- '.github/workflows/lint-i18n.yml' | |
branches: | |
- main | |
pull_request: | |
paths: | |
- '**.js' | |
- '**.cjs' | |
- '**.ts' | |
- '**.tsx' | |
- '**/package.json' | |
- 'package-lock.json' | |
- 'web-stories.php' | |
- 'includes/**.php' | |
- '.github/workflows/lint-i18n.yml' | |
permissions: | |
contents: read | |
# Cancels all previous workflow runs for pull requests that have not completed. | |
concurrency: | |
# The concurrency group contains the workflow name and the (target) branch name. | |
group: ${{ github.workflow }}-${{ github.head_ref || github.ref }} | |
cancel-in-progress: true | |
jobs: | |
lint: | |
name: Lint | |
runs-on: ubuntu-latest | |
timeout-minutes: 10 | |
steps: | |
- name: Harden Runner | |
uses: step-security/harden-runner@8ca2b8b2ece13480cda6dacd3511b49857a23c09 | |
with: | |
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs | |
- name: Checkout | |
uses: actions/checkout@8ade135a41bc03ea155e62e844d188df1ea18608 | |
- name: Install WP-CLI | |
run: | | |
curl -O https://raw.githubusercontent.com/wp-cli/builds/gh-pages/phar/wp-cli.phar | |
chmod +x wp-cli.phar | |
mkdir -p bin | |
mv wp-cli.phar bin/wp | |
echo "${PWD}/bin" >> $GITHUB_PATH | |
- name: WP-CLI Info | |
run: wp cli info | |
- name: Install latest version of i18n-command | |
run: wp package install wp-cli/i18n-command:dev-main | |
- name: List packages | |
run: wp package list | |
- name: Setup Node | |
uses: actions/setup-node@5e21ff4d9bc1a8cf6de233a3057d20ec6b3fb69d | |
with: | |
node-version-file: '.nvmrc' | |
cache: npm | |
- name: Setup PHP | |
uses: shivammathur/setup-php@7fdd3ece872ec7ec4c098ae5ab7637d5e0a96067 | |
with: | |
php-version: '8.0' | |
coverage: none | |
tools: composer | |
- name: Install dependencies | |
run: | | |
npm ci | |
env: | |
PUPPETEER_SKIP_DOWNLOAD: true | |
- name: Install PHP dependencies | |
uses: ramsey/composer-install@83af392bf5f031813d25e6fe4cd626cdba9a2df6 | |
with: | |
composer-options: '--prefer-dist --no-progress --no-interaction' | |
- name: Setup Bun | |
uses: oven-sh/setup-bun@bd26ebd2c71c1f353895b3992d78cafe0a0eaf91 | |
with: | |
bun-version: '0.5.9' | |
- name: Build plugin | |
run: bun run build:js | |
- name: Bundle regular version | |
run: bun run workflow:build-plugin | |
# Check if as many strings as expected were found. | |
# Fail job if `wp i18n make-pot` returns any warnings. | |
# Some false positive warnings are removed due to a bug in the string extraction. | |
# That's why this step is unfortunately a bit more complex. | |
# See https://github.com/wp-cli/i18n-command/issues/154 | |
- name: Generate POT file | |
run: | | |
OUTPUT=$((wp i18n make-pot build/web-stories build/web-stories.pot) 2>&1 >/dev/null) | |
HAS_ERROR=false | |
EXPECTED_NUMBER_OF_STRINGS=1000 | |
NUMBER_OF_FOUND_STRINGS=$(grep -o msgstr build/web-stories.pot | wc -l | xargs) | |
if (( "$NUMBER_OF_FOUND_STRINGS" < "$EXPECTED_NUMBER_OF_STRINGS" )); then | |
HAS_ERROR=true | |
echo "String extraction found only $NUMBER_OF_FOUND_STRINGS translatable strings. Expected at least $EXPECTED_NUMBER_OF_STRINGS." | |
fi | |
IFS=$'\n' | |
declare -a WARNINGS=($OUTPUT) | |
unset IFS | |
for WARNING in "${WARNINGS[@]}"; do | |
# Filter false positives. | |
if [[ $WARNING == *"translator comment"* ]] && [[ $WARNING != *"%s"* ]]; then | |
continue | |
fi | |
HAS_ERROR=true | |
echo $WARNING | |
done | |
if [[ "$HAS_ERROR" = true ]]; then | |
exit 1 | |
fi |