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
ci: use wp-cli i18n command instead of pb-cli #81
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #81 +/- ##
=========================================
Coverage 82.68% 82.68%
Complexity 103 103
=========================================
Files 3 3
Lines 283 283
=========================================
Hits 234 234
Misses 49 49 Continue to review full report at Codecov.
|
…s-sso into feat/wp-i18n
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks Ned
Partial resolution to https://github.com/pressbooks/private/issues/910. In addition to replacing the
wp pb make-pot
command withwp i18n make-pot
, this PR also replaces theupdate-translations.yml
workflow's use of gettext with thewp i18n make-mo
command, which does the same thing.I've commented out the step which autocommits changes. Instead, when these workflows run if they result in any changed files, they will open a PR. Devs can merge if it looks good. In future, you may wish to switch back to autocommit.