Update imports.md

[manojo]

Without the additional braces making the SimpleRef. optional in ImportExpr, an expression such as import java as j wouldn't technically be accepted by the grammar specification.

Co-authored-by: @keynmol

[julienrf]

Thank you @manojo for the PR! I think if we change it here, we should also change it in the syntax.md file (https://github.com/lampepfl/dotty/blob/main/docs/_docs/reference/syntax.md) and in https://github.com/lampepfl/dotty/blob/main/docs/_docs/internals/syntax.md.

But before we go further, I would like @odersky to approve this change.

[odersky]

No, this case is already covered by the second production for ImportExpr. The proposed change is not what we want or implement.

                    |  SimpleRef ‘as’ id

[manojo]

Suggested change

	ImportExpr ::= { SimpleRef {‘.’ id} ‘.’ } ImportSpec
	ImportExpr ::= SimpleRef {‘.’ id} ‘.’ ImportSpec
	| SimpleRef `as` id

[odersky]

Yes, that's the right fix.

[manojo]

@odersky thank you for the message. It looks like the reference syntax is indeed correct, but the imports.md file does not currently have the second production for ImportExpr. @julienrf I've tried to add the second production as a suggestion to this PR. Let me know if this makes sense :-)

[julienrf]

Yes, thank you Mano it makes sense! Could you please apply the changes (I can’t do it from GitHub) and re-open the PR?

[odersky]

Thanks for the fix, Mano!