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

Wrong scope for dependencies in maven pom file io.arrow-kt:arrow-fx-kotlinx-coroutines:0.10.5 #139

Closed
routis opened this issue Apr 5, 2020 · 2 comments · Fixed by #140
Closed
Assignees

Comments

@routis
Copy link

routis commented Apr 5, 2020

The newly released io.arrow-kt:arrow-fx-kotlinx-coroutines:0.10.5 doesn't seem to have a correct pom file.
It declares dependencies to three libraries:

  • io.arrow-kt:arrow-fx:0.10.5
  • org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.3.61
  • org.jetbrains.kotlinx:kotlinx-coroutines-core:1.3.3
    Unfortunately those 3 dependencies are declared in scope "runtime" which should have been "compile". So, this means that a project that depends on arrow-fx-kotlinx-coroutines needs to re-declare those dependencies in the correct scope.
@rachelcarmena rachelcarmena transferred this issue from arrow-kt/arrow Apr 6, 2020
@rachelcarmena
Copy link
Member

Thanks @routis for raising this issue 🙌

They aren't declared in the right scope:

<dependency>
<groupId>io.arrow-kt</groupId>
<artifactId>arrow-fx</artifactId>
<version>0.10.5</version>
<scope>runtime</scope>
</dependency>
<dependency>
<groupId>org.jetbrains.kotlin</groupId>
<artifactId>kotlin-stdlib-jdk7</artifactId>
<version>1.3.61</version>
<scope>runtime</scope>
</dependency>
<dependency>
<groupId>org.jetbrains.kotlinx</groupId>
<artifactId>kotlinx-coroutines-core</artifactId>
<version>1.3.3</version>
<scope>runtime</scope>
</dependency>

rachelcarmena added a commit that referenced this issue Apr 6, 2020
Fix #139 issue about the wrong scope for arrow-fx-kotlinx-coroutines dependencies.
rachelcarmena added a commit that referenced this issue Apr 6, 2020
Co-authored-by: Alberto Ballano <aballano@users.noreply.github.com>
@rachelcarmena
Copy link
Member

It's already fixed for 0.11.0-SNAPSHOT. View last POM file in OSS repository:

https://oss.jfrog.org/artifactory/oss-snapshot-local/io/arrow-kt/arrow-fx-kotlinx-coroutines/0.11.0-SNAPSHOT/

Thanks again @routis 🙌

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 a pull request may close this issue.

2 participants