Unexpected exit behavior for verify and publish tasks

[solarmosaic-kflorence]

The verify and publish tasks use sys.exit(0) on success, and sys.exit(1) on failure. This works fine when you run the command in isolation (e.g. sbt pactPublish), but if you try to run them as part of a sequence of events, either with addCommandAlias or Def.sequential, the sys.exit causes the process to end without running subsequent tasks. For example:

addCommandAlias("publishAll", ";pactPublish;publish;publishEcr")

Will only run pactPublish and then exit, even on success. Using sys.exit(1) on failure seems fine, but on success sys.exit should not be used at all.

See:

• https://github.com/ITV/scala-pact/blob/85978dfea203df47121196531579567b2a70f34d/sbt-scalapact-shared/src/main/scala/com/itv/scalapact/plugin/shared/ScalaPactPublishCommand.scala#L65
• https://github.com/ITV/scala-pact/blob/7e1ef1244ad584e906b115330d5514000cf7f812/sbt-scalapact-shared/src/main/scala/com/itv/scalapact/plugin/shared/ScalaPactVerifyCommand.scala#L77