Rename reflect/reify and make them plain methods

[nicolasstucki]

Remove implicit conversion of the extension methods from the API

[allanrenucci]

Is that what we decided? I thought it was:

(exp: Expr[T]).toTree
(tpe: Type[T]).toTypeTree
(tree: Tree).toExpr[T]

An then replace the extension method toExpr on Any by a normal method

[nicolasstucki]

Didn't we said that that was not good because both methods would still be called toExpr?

[odersky]

I believe the .toExpr on Any is actually the more important of the two so I would leave it.

But it would be good to have extension methods for converting between Expr[T] and Tree. What about this one:

def (e: Expr[T]) unseal[T]: Tree
def (t: Tree) seal[T]: Expr[T]

[smarter]

I believe the .toExpr on Any is actually the more important of the two so I would leave it.

Is it really ? It means that every time you press "." in the IDE it will show up, even though 99% it won't be what you're looking for. Not to mention that this means you'll have a hard time calling it on any class that defines its own toExpr.

Didn't we said that that was not good because both methods would still be called toExpr?

No, because if it's not an extension method you give it another name, like liftToExpr

[nicolasstucki]

Is it really ? It means that every time you press "." in the IDE it will show up, even though 99% it won't be what you're looking for. Not to mention that this means you'll have a hard time calling it on any class that defines its own toExpr.

Only if you imported scala.quoted._

[nicolasstucki]

As discussed in the meeting, we will use

def (e: Expr[T]) unseal[T]: Tree
def (t: Tree) seal[T]: Expr[T]

[allanrenucci]

Can you update the minitest in the community build as well

[nicolasstucki]

I fixed minitest in the community build and it will not be affected further by this change.