Drop coursier '--scala-version' when launching bloop console

[fooblahblah]

Fixes #1301 (bloop console exits with ClassNotFound exception).

Note, I manually tested 2.13 and 2.12 with this change. I'm not sure this ever worked for 2.11 or 2.10?

[jvican]

LGTM! Thanks for the contribution and looking into this! 👍 I remember this was not working for 2.11 in the past but seems like the bug we run into is fixed in the latest release and we can do without this argument.

[alexarchambault]

I'm kind of surprised --scala-version has to be removed… I would actually expect things to fail sometimes without it (because this can bring artifacts corresponding to different full scala versions). Did you compare the class paths you get with and without the --scala-version? You can get these class paths by passing -v -v to launch, or using fetch instead of launch (and removing the --main-class argument).

[jvican]

Do you recommend reverting this change and looking into what's happening @alexarchambault? I'm concerned here that not using --scala-version might disrupt the ammonite console integration for other users after reading your comment.

[alexarchambault]

Do you recommend reverting this change and looking into what's happening?

I don't know if it should be reverted right now (it seems this fixes things in some cases), but maybe be prepared to revert it if anyone complains?

It'd be nice to look more closely at what happened, yes. Like looking at the arguments passed to coursier and the class path it computed, and comparing with and without --scala-version.