lock: do not emit version for direct URLs

It could be dynamic, so not emitting it is a better default. In the future we could consider emitting it when we know it is not dynamic.

Author: sbidoul

src/pip/_internal/models/pylock.py

@@ -84,10 +84,7 @@ def from_install_requirement(cls, ireq: InstallRequirement) -> Self:
         dist = ireq.get_dist()
         download_info = ireq.download_info
         assert download_info
-        package = cls(
-            name=dist.canonical_name,
-            version=str(dist.version),
-        )
+        package = cls(name=dist.canonical_name)
         package.direct = ireq.is_direct if ireq.is_direct else None
         if package.direct:
             if isinstance(download_info.info, VcsInfo):
@@ -120,6 +117,7 @@ def from_install_requirement(cls, ireq: InstallRequirement) -> Self:
                 # should never happen
                 raise NotImplementedError()
         else:
+            package.version = str(dist.version)
             if isinstance(download_info.info, ArchiveInfo):
                 if not download_info.info.hashes:
                     raise NotImplementedError()