rename ExpressionWalker.traverse{Shape,TripleConstraint}s ?

[ericprud]

Version

extends-validation

Question

I think the rationale for the
ExpressionWalker.traverse{Shape,TripleConstraint}s names is that traversing a Shape means you walk the

• TripleExpressions in that Shape,
• ValueExpressions in that TripleConstraint.
  That said, I feel like traverse{TripleConstraint,ValueExpression}s might be more intuitive.

If we want the name to follow the ShExJ attributes, they are Shape.expression and TripleConstraint.valueExpr respectively.