https://pantsbuild.org/ logo
Join Slack
Powered by
Can someone help me understand a few lines from `s...
# development
b
Can someone help me understand a few lines from 
src/python/pants/backend/scala/dependency_inference/rules.py
? Specifically:
Copy code
if not target.has_field(JvmResolveField):
        return InjectedDependencies()

    scala_plugins = await Get(
        ScalaPluginTargetsForTarget, ScalaPluginsForTargetWithoutResolveRequest(target)
    )
Why are we using 
ScalaPluginsForTargetWithoutResolveRequest
if the target does have a resolve? (as evidenced by the confition?)
f
oversight? cc @ancient-vegetable-10556 since 
git
annotates that change to you
a
👀
b
This will be changed slightly in an upcoming PR, but the core of the issue will remain
a
one moment
I don’t have good recollection of this work, it was several months ago HOWEVER
the naming might be a bit obtuse here. What you are seeing is scala source targets do have a resolve, but the plugins probably should not. You associate the scala sources with plugins, so that they’re flagged as dependencies
The request might better be named 
ScalaPluginsWithoutResolveForTargetRequest
, maybe
b
Ah then my change isn't breaking any assumption
👍 thanks yall
a
Right
Now that my memory is returning on this: All scala targets have resolves
2 Views
Open in Slack
PreviousNext
Powered by