Merge pull request #3760 from mpollmeier/mpollmeier/resolver-not-needed-if-not-publishing

don't require publishTo specified if publishArtifact is `false`

main/src/main/scala/sbt/Defaults.scala

@@ -1937,8 +1937,10 @@ object Classpaths {
         if (isSnapshot.value) "integration" else "release",
         ivyConfigurations.value.map(c => ConfigRef(c.name)).toVector,
         packagedArtifacts.in(publish).value.toVector,
-        checksums.in(publish).value.toVector,
-        getPublishTo(publishTo.value).name,
+        checksums.in(publish).value.toVector, { //resolvername: not required if publishTo is false
+          val publishToOption = publishTo.value
+          if (publishArtifact.value) getPublishTo(publishToOption).name else "local"
+        },
         ivyLoggingLevel.value,
         isSnapshot.value
       )

notes/1.2.0/publishTo-not-required-if-not-publishing.md

@@ -0,0 +1,12 @@
+
+[@mpollmeier]: https://github.com/mpollmeier
+
+[#3760]: https://github.com/sbt/sbt/pull/3760
+
+### Fixes with compatibility implications
+
+### Improvements
+
+### Bug fixes
+
+- Remove requirement on `publishTo` if `publishArtifact` is false. [#3760][] by [@mpollmeier][]

sbt/src/sbt-test/project/no-publish/build.sbt

@@ -0,0 +1,3 @@
+// this is supposed to do nothing, and more importantly: not fail
+// https://github.com/sbt/sbt/pull/3760
+publishArtifact := false

sbt/src/sbt-test/project/no-publish/test

@@ -0,0 +1 @@
+> publish