Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ci/phpstan #695

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Ci/phpstan #695

wants to merge 3 commits into from

Conversation

nickvergessen
Copy link
Member

$ composer run phpstan -- --verbose
> vendor/bin/phpstan analyse -c phpstan.neon '--verbose'
 9/9 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%  1 sec

 ------ --------------------------------------------------------------------------------------------------------------------------------------- 
  Line   Controller/APIController.php                                                                                                           
 ------ --------------------------------------------------------------------------------------------------------------------------------------- 
  71     Parameter #1 $data of class OCP\AppFramework\Http\DataResponse constructor expects T of OCP\AppFramework\Http\DataResponseType,        
         list<array<string, int|true>> given.                                                                                                   
         🪪  argument.type                                                                                                                      
  78     Parameter #1 $data of class OCP\AppFramework\Http\DataResponse constructor expects T of OCP\AppFramework\Http\DataResponseType, array  
         given.                                                                                                                                 
         🪪  argument.type                                                                                                                      
  82     Parameter #1 $data of class OCP\AppFramework\Http\DataResponse constructor expects T of OCP\AppFramework\Http\DataResponseType, array  
         given.                                                                                                                                 
         🪪  argument.type                                                                                                                      
  98     Parameter #1 $data of class OCP\AppFramework\Http\DataResponse constructor expects T of OCP\AppFramework\Http\DataResponseType,        
         array<string, int|true> given.                                                                                                         
         🪪  argument.type                                                                                                                      
  121    Parameter #1 $data of class OCP\AppFramework\Http\DataResponse constructor expects T of OCP\AppFramework\Http\DataResponseType, array  
         given.                                                                                                                                 
         🪪  argument.type                                                                                                                      
  133    Parameter #1 $data of class OCP\AppFramework\Http\DataResponse constructor expects T of OCP\AppFramework\Http\DataResponseType, array  
         given.                                                                                                                                 
         🪪  argument.type                                                                                                                      
 ------ --------------------------------------------------------------------------------------------------------------------------------------- 


                                                                                                                        
 [ERROR] Found 6 errors                                                                                                 
                                                                                                                        

Elapsed time: 1 second
Used memory: 92 MB

@nickvergessen nickvergessen removed the request for review from max-nextcloud January 2, 2025 14:14
@nickvergessen nickvergessen marked this pull request as draft January 2, 2025 14:14
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.

1 participant