Skip to content
This repository has been archived by the owner on Feb 24, 2021. It is now read-only.

Bump kotest #188

Closed
wants to merge 40 commits into from
Closed

Bump kotest #188

wants to merge 40 commits into from

Conversation

aballano
Copy link
Member

No description provided.

Copy link
Member

@rachelcarmena rachelcarmena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aballano , thanks!! In summary: I'd just upgrade arrow-core-test which is using KotlinTest now as a compile dependency and Gradle finds and replaces arrow-core dependency by project dependency. That shouldn't happen in the rest of libraries where just removing exclude Arrow should work.

build.gradle Outdated
Comment on lines 33 to 37
tasks.withType(org.jetbrains.kotlin.gradle.tasks.KotlinCompile).all {
kotlinOptions {
jvmTarget = "1.8"
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome job in this PR 👏 Please, could you locate these lines in arrow repository? subprojects section in generic-conf.gradle. JAVA_VERSION is available. Thanks!!

@rachelcarmena
Copy link
Member

@aballano, the checks are going to find https://github.com/arrow-kt/arrow-fx/tree/ab/bump-kotest branch because arrow-kt/arrow-fx#230 is open. Close that PR in order not to consider that branch during checks (I think it's not fixed yet).

@@ -1,9 +1,6 @@
@file:Suppress("NAME_SHADOWING")
package arrow.core

inline fun <A, R> mapN(a: A?, fn: (A) -> R): R? =
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why these changes unrelated to PR?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# Conflicts:
#	arrow-core-data/src/test/kotlin/arrow/core/IorTest.kt
#	arrow-core-test/src/main/kotlin/arrow/core/test/generators/Generators.kt
@nomisRev
Copy link
Member

@aballano Can we consider this ticket in TODO with this PR?

aballano and others added 10 commits August 12, 2020 17:22
# Conflicts:
#	arrow-core-data/src/test/kotlin/arrow/core/BooleanTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/ConstTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/IdTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/IorTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/ListKTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/NonEmptyListTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/OptionTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/SequenceKTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/ValidatedTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/extensions/NumberInstancesTest.kt
#	arrow-core-data/src/test/kotlin/arrow/core/extensions/StringInstancesTest.kt
# Conflicts:
#	arrow-core-test/src/main/kotlin/arrow/core/test/generators/GenK.kt
#	arrow-core-test/src/main/kotlin/arrow/core/test/generators/Generators.kt
#	arrow-core-test/src/main/kotlin/arrow/core/test/laws/HashLaws.kt
@aballano aballano marked this pull request as draft August 18, 2020 13:07
@aballano aballano closed this Jan 7, 2021
@aballano aballano deleted the ab/bump-kotest branch January 7, 2021 13:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants