apollo-kotlin fails when using refreshVersions

• [ ✔️ ] I have read the guidelines for submitting issues

🐛 Describe the bug

Apollo checks to make sure all versions are equal: https://github.com/apollographql/apollo-kotlin/blob/a1536a269ffe5896f1b5290e55bd361e2d398861/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo3/gradle/internal/DefaultApolloExtension.kt#L167

This check fails when using refreshVersions.

This can be worked around by using versionFor in the dependencies.

⚠️ Current behavior

This results in the generateApolloSources task giving the following error:

Apollo: All apollo versions should be the same. Found:
  [3.1.0, _]

✅ Expected behavior

The version check to succeed.

💣 Steps to reproduce

Configuration:

buildscript {
    ...
    dependencies {
        ...
        classpath("com.apollographql.apollo3:apollo-gradle-plugin:_")
    }
}

plugins {
    ...
    id("com.apollographql.apollo3")
}

dependencies {
    implementation("com.apollographql.apollo3:apollo-runtime:_")
    implementation("com.apollographql.apollo3:apollo-adapters:_")
}

[imashnake0]

Fixed in 3.2.0 🎉.

[martinbonnin]

Thinking more about this, I’m having second thoughts… Resolving a configuration isn’t something trivial and I’m too afraid of potential side effects. Also, if for some reason, another unrelated dependency fails to resolve, this will show up with “checkApolloVersions” in the error, which could be misleading.

~I’m ok disabling the check if de.fayard.refreshVersions is applied (or maybe just filtering out ‘'). What do you think?~ Edit: never mind, this is what you said in your comment above. I’m just going to filter out '’ as a pragmatic first step. For disabling the check, see comment above.