Prevent align rules from trying to align project dependencies. Adding a version causes Gradle to attempt to resolve an artifact

Author: DanielThomas

CHANGELOG.md

@@ -1,5 +1,6 @@
 1.3.0 / 2016/04/28
 ==================
+- BUGFIX: Align rules attempt to align project dependencies, causing them to be resolved as remote artifacts
 - Rules files may now be optional, so optionated rules aren't applied without users opting in
 - Align rules support regular expressions in the group, includes and excludes
 - Empty rules types can be excluded from rules files (required for backwards compatibility with old rules files, but also makes working with them nicer)

src/functionalTest/groovy/nebula/plugin/resolutionrules/AlignRulesMultiprojectSpec.groovy

@@ -94,7 +94,9 @@ class AlignRulesMultiprojectSpec extends IntegrationSpec {
         def results = runTasksSuccessfully(':b:dependencies', '--configuration', 'compile')
 
         then:
-        noExceptionThrown()
+        results.standardOutput.contains("""compile - Dependencies for source set 'main'.
+\\--- project :a
+""")
     }
 
     def 'cycle like behavior'() {

src/main/groovy/nebula/plugin/resolutionrules/Rules.groovy

@@ -19,6 +19,7 @@ import org.gradle.api.Project
 import org.gradle.api.artifacts.*
 import org.gradle.api.artifacts.component.ComponentSelector
 import org.gradle.api.artifacts.component.ModuleComponentIdentifier
+import org.gradle.api.artifacts.component.ProjectComponentIdentifier
 import org.gradle.api.internal.artifacts.ivyservice.ivyresolve.strategy.DefaultVersionComparator
 import org.gradle.api.specs.Specs
 import org.joda.time.DateTime
@@ -215,7 +216,7 @@ class AlignRules implements ProjectConfigurationRule {
 
         def copy = configuration.copyRecursive()
         copy.exclude group: project.group, module: project.name
-        def artifacts
+        Set<ResolvedArtifact> artifacts
         def resolvedConfiguration = copy.resolvedConfiguration
         if (resolvedConfiguration.hasError()) {
             def lenientConfiguration = resolvedConfiguration.lenientConfiguration
@@ -225,7 +226,10 @@ class AlignRules implements ProjectConfigurationRule {
             artifacts = resolvedConfiguration.resolvedArtifacts
         }
 
-        def moduleVersions = artifacts.collect { it.moduleVersion }
+        def moduleVersions = artifacts.findAll {
+            // Exclude project artifacts from alignment
+            !(it.id.componentIdentifier instanceof ProjectComponentIdentifier)
+        }.collect { it.moduleVersion }
         def selectedVersion = [:]
         aligns.each { AlignRule align ->
             if (align.shouldNotBeSkipped(extension)) {