Skip to content

Commit

Permalink
Merge pull request #3241 from scala-steward-org/topic/fail-on-vcs-opt…
Browse files Browse the repository at this point in the history
…ions

Fail on `--vcs-*` options
  • Loading branch information
fthomas authored Jan 22, 2024
2 parents bd735d9 + b7ef418 commit 33d3ad7
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ object Cli {
"vcs-type",
s"deprecated in favor of --${name.forgeType}",
visibility = Visibility.Partial
)
).validate(s"--vcs-type is deprecated; use --${name.forgeType} instead")(_ => false)

private val forgeType = {
val help = ForgeType.all.map(_.asString).mkString("One of ", ", ", "") +
Expand All @@ -120,7 +120,7 @@ object Cli {
"vcs-api-host",
s"deprecated in favor of --${name.forgeApiHost}",
visibility = Visibility.Partial
)
).validate(s"--vcs-api-host is deprecated; use --${name.forgeApiHost} instead")(_ => false)

private val forgeApiHost: Opts[Uri] =
option[Uri](name.forgeApiHost, s"API URL of the forge; default: ${GitHub.publicApiBaseUrl}")
Expand All @@ -132,7 +132,7 @@ object Cli {
"vcs-login",
s"deprecated in favor of --${name.forgeLogin}",
visibility = Visibility.Partial
)
).validate(s"--vcs-login is deprecated; use --${name.forgeLogin} instead")(_ => false)

private val forgeLogin: Opts[String] =
option[String](name.forgeLogin, "The user name for the forge").orElse(vcsLogin)
Expand Down

0 comments on commit 33d3ad7

Please sign in to comment.