Skip to content

Conversation

@kenjis
Copy link
Member

@kenjis kenjis commented Apr 7, 2024

No description provided.

kenjis and others added 30 commits February 27, 2024 17:22
We should use $options['component'] because it takes precedence.
See the code below to create a shared instance.
feat: add spark command to check php.ini
 Conflicts:
	system/Test/TestResponse.php
refactor: remove deprecated BaseService::discoverServices()
 Conflicts:
	tests/_support/Log/Handlers/TestHandler.php
refactor: remove unnecessary BaseService::$services assignment
kenjis added 26 commits April 7, 2024 12:38
vendor/bin/phpunit --migrate-configuration -c phpunit.xml.dist
Message:  file_put_contents(/home/runner/work/CodeIgniter4/CodeIgniter4/build/coverage/text/coverage.txt): Failed to open stream: No such file or directory
Location: /home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/src/Runner/CodeCoverage.php:317

Script done.
Error: Process completed with exit code 255.
1) CodeIgniter\Database\DatabaseTestCase\DatabaseTestCaseMigrationOnce1Test::testMigrationDone
TypeError: json_encode(): Argument #2 ($flags) must be of type int, bool given

/home/runner/work/CodeIgniter4/CodeIgniter4/system/Test/Constraints/SeeInDatabase.php:118
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Test/DatabaseTestTrait.php:282
/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Database/DatabaseTestCase/DatabaseTestCaseMigrationOnce1Test.php:86
There was 1 error:

 1) CodeIgniter\Security\SecurityCSRFSessionRandomizeTokenTest::testCSRFVerifyPostInvalidToken
PHPUnit\Framework\Exception: PHP Warning:  hex2bin(): Hexadecimal input string must have an even length in /home/runner/work/CodeIgniter4/CodeIgniter4/system/Security/Security.php on line 378
PHP Warning:  hex2bin(): Hexadecimal input string must have an even length in /home/runner/work/CodeIgniter4/CodeIgniter4/system/Security/Security.php on line 378
There was 1 error:

1) CodeIgniter\CommonFunctionsTest::testTraceWithCSP
Test was run in child process and ended unexpectedly
…sting

docs: fix merge mistake about "Deployment to Shared Hosting Services"
@kenjis kenjis mentioned this pull request Apr 7, 2024
5 tasks
@kenjis kenjis merged commit c4a89e1 into master Apr 7, 2024
@kenjis kenjis mentioned this pull request Apr 14, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants