I get this when running drush dmu-upgrade diff on the latest master/pharborist.

Let's get some test coverage for this.

Comments

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented

Agreed. fromScalar() was removed from Pharborist (it's still there, just renamed...I think it's now Node::fromValue()), but all the function modifiers need tests.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs review

Quick-fixed in master, and tested against diff.

  • phenaproxima committed 4084544 on master 
    Issue #2351047: Fatal error: Call to undefined method Pharborist\Node::...
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Fixed

Yep, looks good.

  • webchick committed 47b97e3 on 8.x-1.x authored by phenaproxima 
    Issue #2351047: Fatal error: Call to undefined method Pharborist\Node::...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.