Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix #176: Add Paradox directive to show binding build tool integration #186

Merged
merged 1 commit into from
Sep 7, 2018

Conversation

jonas
Copy link
Member

@jonas jonas commented Sep 4, 2018

screen shot 2018-09-03 at 22 46 14

screen shot 2018-09-03 at 22 46 21

screen shot 2018-09-03 at 22 46 27


def renderBindingDependency(binding: String, printer: Printer): Unit = {
val artifactName = s"lib$binding"
val artifactId = s"${artifactName}_native0.3_${ScalaBinaryVersion}"
Copy link
Member Author

Choose a reason for hiding this comment

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

native0.3 is currently hard-coded. I don't know if it is possible to get it from the sbt-scala-native plugin.

@jonas jonas force-pushed the binding-artifact-id branch from 2ecadf1 to 2118de1 Compare September 4, 2018 02:50
@jonas jonas requested a review from kornilova203 September 4, 2018 02:50
@jonas jonas force-pushed the binding-artifact-id branch from 2118de1 to 0351ff5 Compare September 4, 2018 04:54
Copy link
Member

@kornilova203 kornilova203 left a comment

Choose a reason for hiding this comment

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

Awesome

artifact="libiconv_$scala.binary.version$"
version="$project.version$"
}
@@binding[iconv]
Copy link
Member

Choose a reason for hiding this comment

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

❤️

@kornilova203
Copy link
Member

Closes #176

@kornilova203 kornilova203 merged commit c42019e into master Sep 7, 2018
@kornilova203 kornilova203 deleted the binding-artifact-id branch September 7, 2018 19:16
@jonas
Copy link
Member Author

jonas commented Sep 7, 2018

Thanks for the review, I will work on releasing version 0.3.1.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants