Skip to content

updated sbt configuration, added cross publish#8

Open
atais wants to merge 1 commit intorklaehn:masterfrom
atais:master
Open

updated sbt configuration, added cross publish#8
atais wants to merge 1 commit intorklaehn:masterfrom
atais:master

Conversation

@atais
Copy link

@atais atais commented Jun 27, 2018

Hi.

This is not ready to go live, since first there is a missing dependency on your other project

[error] (coreJS/*:update) sbt.ResolveException: unresolved dependency: com.rklaehn#sonicreducer_sjs0.6_2.11;0.5.0: not found
[error] (coreJVM/*:update) sbt.ResolveException: unresolved dependency: com.rklaehn#sonicreducer_2.11;0.5.0: not found

I could help with the other as well, or you could just add the enableCrossBuild = true in the release steps. That should do.

In the second step I would update the cats dependency.
I have a small issue there, that maybe you could help me out with later.

How does it look like?

@atais
Copy link
Author

atais commented Jun 27, 2018

I used my project with cross publish as a reference
https://github.com/atais/Fixed-Length

@codecov-io
Copy link

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #8   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           6      6           
  Lines         371    371           
  Branches       96     96           
=====================================
  Hits          371    371

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 86dcada...181a454. Read the comment docs.

@jho
Copy link

jho commented Sep 16, 2019

Any chance of merging this in? I am trying to using radixtree in Spark (so still on 2.11) and could really use this as well. I am willing to help if needed.

@atais
Copy link
Author

atais commented Sep 16, 2019

it was not that easy as I first thought, but you can try.
it requires changes in one of the dependencies though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments