Andrew Taylor

> 99-out-of-100 times, the reason a developer is adding a file to their theme is because they need it there. That is an assumption based on your workflow, I don't...

@benoitchantre and I are talking through what reversing the approach and maintaining a blacklist instead of a whitelist for files to copy from the dev theme to the production theme....

> Let me look at this before merge. I too have modified this functionality for a client. Let me compare this with my code. @bamadesigner did you have a chance...

@justlevine out of curiosity did you read [the high-level PHP documentation for v2](https://github.com/wprig/docs/blob/master/documentation/php.md)? If so, what additional PHP documentation do you think is needed? If not, would it have helped...

> apologies - I wrote a lot No need to apologize, this is the type of feedback we need to improve WP Rig and the docs. It is (understandably) tough...

@benoitchantre I think `export.FilesToCopy` in the config will work here. It takes an array of files and copies all of them as-is to the production bundle. Something like this in...

> I think it should be part of `config.default.json` as it is a requirement from the Theme Review Team. I respectfully disagree. We can document how to make sure the...

> .mo files should not be copied to the prod theme but should be generated from the rewritten .po files in the production as a last step I think [`gettext-parser`](https://www.npmjs.com/package/gettext-parser)...

`wp i18n make-json languages --no-purge` isn't working in the gulp translate task. I've also called it manually with `.po` files in languages and it didn't work as expected. Aside from...

For the tests, I tried using [the `beforeAll` and `afterAll` feature in Jest](https://jestjs.io/docs/en/setup-teardown.html) to do some set up (e.g. copying the `.po` and `.mo` files ) and running the gulp...