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

add conflict with phpstan 0.12.26 and 0.12.27 #805

Closed
wants to merge 1 commit into from

Conversation

dmaicher
Copy link
Contributor

@dmaicher dmaicher commented Jun 8, 2020

PHPstan switched to AST based static analysis and there seem to be some issues with that.

The fail is actually wrong:

 ------ ---------------------------------------------------------------------------------- 
  Line   src/Component/Facet/JsonRange.php                                                 
 ------ ---------------------------------------------------------------------------------- 
  38     Call to an undefined method Solarium\Component\Facet\JsonRange::jsonFacetInit().  
 ------ ---------------------------------------------------------------------------------- 

                                                                                                                        
 [ERROR] Found 1 error   

Reported the bug: phpstan/phpstan#3415

@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2020

Codecov Report

Merging #805 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #805   +/-   ##
=========================================
  Coverage     90.61%   90.61%           
  Complexity     3530     3530           
=========================================
  Files           304      304           
  Lines          7627     7627           
=========================================
  Hits           6911     6911           
  Misses          716      716           
Flag Coverage Δ Complexity Δ
#unittests 90.61% <ø> (ø) 3530.00 <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a0ab62...723b957. Read the comment docs.

@thomascorthals
Copy link
Member

Thanks!

My PR that fails because of a possible Solr bug now also fails because of a phpstan bug. ;)

@dmaicher dmaicher changed the title add conflict with phpstan 0.12.26 add conflict with phpstan 0.12.26 and 0.12.27 Jun 9, 2020
@@ -43,5 +43,8 @@
"psr-4": {
"Solarium\\Tests\\": "tests/"
}
},
"conflict": {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah actually this is not good I think. It will prevent those versions to be installed for anyone using solarium. We only need this for "dev" here...

@dmaicher
Copy link
Contributor Author

dmaicher commented Jun 9, 2020

For now I guess we can only pin the exact version 0.12.25 until a new fix is released? 😢

@dmaicher dmaicher closed this Jun 9, 2020
@dmaicher dmaicher deleted the phpstan_conflict branch June 9, 2020 11:36
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.

None yet

3 participants