Branch: refs/heads/master Home: https://github.com/phpmyadmin/error-reporting-server Commit: 92899d913ede253b8c21991f3490f59655646da8 https://github.com/phpmyadmin/error-reporting-server/commit/92899d913ede253b8c21991f3490f59655646da8 Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00
Changed paths: M .scrutinizer.yml Log Message: ----------- Send the phpstan and phpcs results to Scrutinizer phpcs-run ran the generic standard, which does not know the templates of CakePHP and reported issues that the project standard does not have. Report the results of the tools the project uses instead, like the phpMyAdmin QA branches do, so the Scrutinizer issues match composer check. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> Commit: 53c3dd9b1bd9f59e851e8221e24aea028e07bb1e https://github.com/phpmyadmin/error-reporting-server/commit/53c3dd9b1bd9f59e851e8221e24aea028e07bb1e Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00 Changed paths: M src/Report.php Log Message: ----------- Declare the properties of the Report class The class extends stdClass and fromObject() copies every key of the payload on the instance, so the properties are not declared and the analyzers report "The property user_os does not exist on App\Report". 78af93a354 made them readable without the curly braces, declare them for the analyzer instead. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> Commit: f8759cf74d2b09d1e74686c07b90831dbef3442c https://github.com/phpmyadmin/error-reporting-server/commit/f8759cf74d2b09d1e74686c07b90831dbef3442c Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00 Changed paths: M phpstan-baseline.neon M src/Model/Table/DevelopersTable.php Log Message: ----------- Set the developer fields through the entity API saveFromGithub() writes the properties on the EntityInterface, which Scrutinizer reports six times as "Accessing full_name on the interface". set() is the documented way and validates the fields. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> Commit: a83970bd5c0f119e978cf6b8e1dcca62622892ec https://github.com/phpmyadmin/error-reporting-server/commit/a83970bd5c0f119e978cf6b8e1dcca62622892ec Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00 Changed paths: M tests/TestCase/Controller/EventsSecurityTest.php Log Message: ----------- Use a docblock the analyzers can read for the signature provider Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> Commit: ebbd9fccc5895fb220b006fd0042acf0358074b0 https://github.com/phpmyadmin/error-reporting-server/commit/ebbd9fccc5895fb220b006fd0042acf0358074b0 Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00 Changed paths: M src/Command/SyncGithubIssueStatesCommand.php Log Message: ----------- Return an error code instead of exiting the command Calling exit kills the whole process, in a web context the command would answer with an empty success response. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> _______________________________________________ Git mailing list -- [email protected] To unsubscribe send an email to [email protected]
