New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace jvm-generate-lockfiles
goal with generate-lockfiles
#14176
Replace jvm-generate-lockfiles
goal with generate-lockfiles
#14176
Conversation
# Rust tests and lints will be skipped. Delete if not intended. [ci skip-rust] # Building wheels and fs_util will be skipped. Delete if not intended. [ci skip-build-wheels]
# Rust tests and lints will be skipped. Delete if not intended. [ci skip-rust] # Building wheels and fs_util will be skipped. Delete if not intended. [ci skip-build-wheels]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two commits are distinct.
artifact_inputs: FrozenOrderedSet[str] | ||
|
||
@classmethod | ||
def from_tool(cls, tool: JvmToolBase) -> JvmLockfileRequest: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that before we were erroring when using this method and the DEFAULT_LOCKFILE was used. That is now instead handled by generate_lockfiles.py
.
class JvmLockfileRequest(LockfileRequest): | ||
artifact_inputs: FrozenOrderedSet[str] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LockfileRequest
already has resolve_name
and lockfile_dest
. So this just augments it with additional info.
Note that this generic type will be used for user lockfiles, too. I haven't yet started merging those.
@rule(desc="Generate JVM lockfile", level=LogLevel.DEBUG) | ||
async def generate_jvm_lockfile( | ||
request: JvmLockfileRequest, | ||
) -> Lockfile: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as before, minus the types changing.
@rule | ||
async def load_jvm_lockfile( | ||
request: JvmLockfileRequest, | ||
) -> CoursierResolvedLockfile: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as before, only the types changed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a ton for doing this!
We now have a consistent mechanism for tool lockfiles.
This means that Python and JVM lockfiles now share the same namespace: