bazelbuild / rules_scala

Scala rules for Bazel
Apache License 2.0
363 stars 275 forks source link

Make scaladoc rule handle transient deps better (and add some tests) #1584

Closed crt-31 closed 4 months ago

crt-31 commented 4 months ago

Description

The main goal of this PR is to fix issue where transient dependencies weren't being handled correctly. This issue manifested itself when there was an external library (i.e. scala_import) that was a transient dependency.

Also added tests for the scaladoc rule.

Motivation

This is needed for the scaladoc rule to work correctly.

I believe this also addresses the issue that #1331 was trying to address.