Include theme.json; patterns/ in translation scan command#3076
Conversation
|
With recent |
|
@retlehs: Latest stable WP CLI appears to work fine with the used commands now, so this can be merged. To make this more robust, |
|
@kellymears: A dedicated command for |
theme.json in translation scan commandtheme.json; patterns/ in translation scan command
|
@retlehs, @kellymears: Current stable |
retlehs
left a comment
There was a problem hiding this comment.
My apologies for the delay, thank you for this update!
It turns out that when using
--includewithwp i18nthetheme.jsonfile and thepatternsdirectoryneed to be explicitly included, too, as those would be skipped otherwise.
Edit:
The latest
wp i18nversion (>=2.4.0) is needed for some bugfixes of otherwise blocking bugs.Until
wpCLI is also newly released with the newwp i18nversion bundled,the latest
wp i18nversion can be installed independently from thewpCLI version by usingwp package install wp-cli/i18n-command:dev-mainand then just using it as usual (
wp i18n).Edit (October 2023): Current stable
wp cli2.8.1contains a recent enoughwp i18n.