- Issue created by @vasike
- Merge request !461Issue #3505528 by vasike: Fix - PHPStan Issues - from methods docs vars... → (Open) created by vasike
- 🇷🇴Romania vasike Ramnicu Valcea
Created a MR for this
Extra: Align StreamedChatMessageIterator iterator type with its Interface.
This affects all StreamedChatMessageIterator - missing "getIterator" ...Anyway - the level 2 for PHPStan - seems to strict for the current dev state.
So, please, let me know when I could revert ... or please feel free to revert that change.
- 🇷🇴Romania vasike Ramnicu Valcea
There is an example of those issues/errors
https://www.drupal.org/project/gemini_provider/issues/3505520 📌 Fix CS warnings/errors for GitLab CI pipelines Active
and its MR
https://git.drupalcode.org/project/gemini_provider/-/merge_requests/14 - 🇬🇧United Kingdom MrDaleSmith
There are failing tests https://git.drupalcode.org/issue/ai-3505528/-/jobs/4303939. The maintainers have mentioned that any code changes that might cause other custom code or contrib modules to receive errors can't go into the module until it has a 2.0 release, so this may need some caution.
- 🇷🇴Romania vasike Ramnicu Valcea
@mrdalesmith as described in the issue and my comments
I intentionallyIncreased (at least temporary for the MR) the PHPStan level 2
.Just to check the issue was solved .. not present in PHPStan issue.
So I updated the MR - rebased and removed the PHPStan level 2 change ..
For "results" please check the previous mentioned related issue and its MR pipeline.