Address the comments in the review

Author: hamzaremmal

.github/workflows/launchers.yml

@@ -1,5 +1,6 @@
 name: Test CLI Launchers on all the platforms
 on:
+  pull_request:
   workflow_dispatch:
 
 jobs:

compiler/src/dotty/tools/MainGenericRunner.scala

@@ -270,7 +270,7 @@ object MainGenericRunner {
 
     val ranByCoursierBootstrap =
       sys.props.isDefinedAt("coursier.mainJar")
-      || sys.props.get("bootstrap.mainClass").filter(_ == "dotty.tools.MainGenericRunner").isDefined
+      || sys.props.get("bootstrap.mainClass").contains("dotty.tools.MainGenericRunner")
 
     val silenced = sys.props.get("scala.use_legacy_launcher") == Some("true")
 

project/Build.scala

@@ -2323,8 +2323,7 @@ object Build {
       settings(scala3PresentationCompilerBuildInfo)
 
     def asDist(implicit mode: Mode): Project = project.
-      enablePlugins(PackPlugin).
-      enablePlugins(RepublishPlugin).
+      enablePlugins(PackPlugin, RepublishPlugin).
       withCommonSettings.
       settings(commonDistSettings).
       dependsOn(

project/Modes.scala

@@ -1,4 +1,4 @@
-import sbt.{Project, ProjectReference, SettingsDefinition, Plugins}
+import sbt.{Project, ProjectReference, SettingsDefinition}
 
 object Modes {
 

project/scripts/buildScalaBinary

@@ -2,7 +2,10 @@
 
 set -e
 
-source $(dirname $0)/cmdTestsCommon.inc.sh
+ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")" >& /dev/null && pwd)/../.."
+
+# set the $DIST_PROJECT and $DIST_DIR variables
+source "$ROOT/bin/common-platform"
 
 # build the scala/scalac/scaladoc binary, where scala is native for the current platform.
 "$SBT" "$DIST_PROJECT/pack"