Results 150 comments of Matt Robinson

As pre review comment on Slack - thanks @xxsimoxx

Is this issue still valid after #265 was merged?

Looking at the [changes](https://github.com/ClassicPress/ClassicPress/pull/265/files) made with #265 the link to the upstream Codex page has been removed and replaced with a more streamlined process. Closing.

Discussed at weekly PR review and agreed for merge.

Personally I don't use `WP-Cli` so this is not my area. One issue I have with the solution is it is always there, all the time. Would it still work...

Has 2 reviews, looks like a useful change for `wp-cli` users of ClassicPress.

Hi @tmconnect On checking at WP it seems these change apply more broadly to other functions in the `general-template.php` file. https://core.trac.wordpress.org/changeset/48370 It would be prudent to backport to the full...

@tmconnect Could you provide a use case for this change? I've done a search at https://wpdirectory.net/ for `get_template_part` and cannot easily see any plugins or themes making use of the...

PHPUnit tests that were failing are now resolved. For consideration - should we introduce a test to check the default setting on install or are we satisfied that the established...

For future work on this issue, please see [Pull Request 1109](https://github.com/ClassicPress/ClassicPress/pull/1109)