Ben Balter

Results 165 comments of Ben Balter

@copilot [34](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082206018/job/54514464086?pr=339#step:6:35) FOUND 1 ERROR AFFECTING 1 LINE [35](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082206018/job/54514464086?pr=339#step:6:36) -------------------------------------------------------------------------------- [36](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082206018/job/54514464086?pr=339#step:6:37) 133 | ERROR | Use placeholders and $wpdb->prepare(); found $query [37](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082206018/job/54514464086?pr=339#step:6:38) | | (WordPress.DB.PreparedSQL.NotPrepared)

@copilot The job failed with exit code 1, but the logs are not explicit about the cause of the error. It appears to be related to PHP 8.4 compatibility, given...

@copilot There were 8 failures: [103](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:104) [104](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:105) 1) CLITest::test_cli_command_class_exists [105](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:106) Jekyll_Export_Command class should exist when WP_CLI is defined [106](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:107) Failed asserting that false is true. [107](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:108) [108](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:109) /home/runner/work/wordpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/tests/test-cli.php:28 [109](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:110) [110](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18697776397/job/53319754513?pr=341#step:9:111)...

@copilot FILE: ...rdpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/tests/test-cli.php [36](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18699585178/job/53325222431?pr=341#step:6:37) -------------------------------------------------------------------------------- [37](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18699585178/job/53325222431?pr=341#step:6:38) FOUND 1 ERROR AFFECTING 1 LINE [38](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18699585178/job/53325222431?pr=341#step:6:39) -------------------------------------------------------------------------------- [39](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18699585178/job/53325222431?pr=341#step:6:40) 25 | ERROR | [x] Whitespace found at end of line [40](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18699585178/job/53325222431?pr=341#step:6:41) | | (Squiz.WhiteSpace.SuperfluousWhitespace.EndLine)...

@copilot run composer install --no-interaction --prefer-dist --optimize-autoloader and commit changes.

@copilot [33](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18699802382/job/53326009556?pr=341#step:9:34) 1) EdgeCasesTest::test_post_with_unicode [34](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18699802382/job/53326009556?pr=341#step:9:35) Attempt to read property "display_name" on false

@copilot 1) EdgeCasesTest::test_copy_recursive_with_symlink [34](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:35) Error: Call to undefined function is_windows() [35](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:36) [36](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:37) /home/runner/work/wordpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/tests/test-edge-cases.php:301 [37](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:38) [38](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:39) 2) IntegrationTest::test_export_multiple_post_types [39](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:40) Attempt to read property "display_name" on false [40](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:41) [41](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:42) /home/runner/work/wordpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/jekyll-exporter.php:155 [42](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/18983401378/job/54407466213?pr=341#step:9:43) /home/runner/work/wordpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/jekyll-exporter.php:277...

@copilot ) IntegrationTest::test_export_special_characters [34](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:35) Attempt to read property "display_name" on false [35](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:36) [36](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:37) /home/runner/work/wordpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/jekyll-exporter.php:155 [37](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:38) /home/runner/work/wordpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/jekyll-exporter.php:277 [38](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:39) /home/runner/work/wordpress-to-jekyll-exporter/wordpress-to-jekyll-exporter/tests/test-integration.php:253 [39](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:40) [40](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:41) 2) WordPressToJekyllExporterTest::test_convert_meta_no_custom_fields [41](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082225345/job/54514422601?pr=341#step:9:42) Attempt to read property "display_name" on false...

@copilot Failing test: [32](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082796602/job/54515670314?pr=341#step:9:33) [33](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082796602/job/54515670314?pr=341#step:9:34) 1) IntegrationTest::test_export_special_characters [34](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082796602/job/54515670314?pr=341#step:9:35) Failed asserting that two strings are equal. [35](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082796602/job/54515670314?pr=341#step:9:36) --- Expected [36](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082796602/job/54515670314?pr=341#step:9:37) +++ Actual [37](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082796602/job/54515670314?pr=341#step:9:38) @@ @@ [38](https://github.com/benbalter/wordpress-to-jekyll-exporter/actions/runs/19082796602/job/54515670314?pr=341#step:9:39) -'Test Post with "Quotes" &...