Skip to content

Commit

Permalink
Refactor Resolver._get_install_requirements() (#1664)
Browse files Browse the repository at this point in the history
Co-authored-by: Sorin Sbarnea <sorin.sbarnea@gmail.com>
  • Loading branch information
atugushev and ssbarnea committed Oct 5, 2022
1 parent 5242ab8 commit 221dd4c
Showing 1 changed file with 5 additions and 10 deletions.
15 changes: 5 additions & 10 deletions piptools/resolver.py
Expand Up @@ -609,14 +609,8 @@ def resolve(self, max_rounds: int = 10) -> Set[InstallRequirement]:
resolver_result = resolver._result
assert isinstance(resolver_result, Result)

# Get reverse requirements from the resolver result graph.
reverse_dependencies = self._get_reverse_dependencies(resolver_result)

# Prepare set of install requirements from resolver result.
result_ireqs = self._get_install_requirements(
resolver_result=resolver_result,
reverse_dependencies=reverse_dependencies,
)
result_ireqs = self._get_install_requirements(resolver_result=resolver_result)

# Filter out unsafe requirements.
if not self.allow_unsafe:
Expand Down Expand Up @@ -676,13 +670,14 @@ def _do_resolve(
return True

def _get_install_requirements(
self,
resolver_result: Result,
reverse_dependencies: Dict[str, Set[str]],
self, resolver_result: Result
) -> Set[InstallRequirement]:
"""Return a set of install requirements from resolver results."""
result_ireqs: Dict[str, InstallRequirement] = {}

# Get reverse requirements from the resolver result graph.
reverse_dependencies = self._get_reverse_dependencies(resolver_result)

# Transform candidates to install requirements
resolved_candidates = tuple(resolver_result.mapping.values())
for candidate in resolved_candidates:
Expand Down

0 comments on commit 221dd4c

Please sign in to comment.