Skip to content

fix: phpstan errors for phpstan-codeigniter (v1.4.0.70400)#898

Merged
datamweb merged 1 commit intocodeigniter4:developfrom
datamweb:fix-new-CI-phpstan
Oct 12, 2023
Merged

fix: phpstan errors for phpstan-codeigniter (v1.4.0.70400)#898
datamweb merged 1 commit intocodeigniter4:developfrom
datamweb:fix-new-CI-phpstan

Conversation

@datamweb
Copy link
Copy Markdown
Collaborator

@datamweb datamweb commented Oct 12, 2023

Description
Explain what you have changed, and why.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@datamweb
Copy link
Copy Markdown
Collaborator Author

@kenjis see here, phpunit /PHP 7.4 - OCI8 passed.🤒

@datamweb datamweb changed the title fix: using the phpstan-codeigniter (v1.4.0.70400) fix: phpstan errors for phpstan-codeigniter (v1.4.0.70400) Oct 12, 2023
@kenjis
Copy link
Copy Markdown
Member

kenjis commented Oct 12, 2023

Oh, yes! It probably fails because Oracle happens to take a long time to start up.
See #899 (comment)

@datamweb
Copy link
Copy Markdown
Collaborator Author

@kenjis thank you!

@datamweb datamweb merged commit 4219d62 into codeigniter4:develop Oct 12, 2023
@datamweb datamweb deleted the fix-new-CI-phpstan branch October 12, 2023 23:26
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.

2 participants