Skip to content

Commit

Permalink
Merge pull request #1 from andriykohut/bugfix/check-all-packages
Browse files Browse the repository at this point in the history
check all packages + fix supported output
  • Loading branch information
andriykohut authored Dec 5, 2023
2 parents 27d67b3 + 4609095 commit 24f15a9
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 6 deletions.
12 changes: 9 additions & 3 deletions src/pdm_readiness/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,15 @@ def add_arguments(self, parser: argparse.ArgumentParser) -> None:

def handle(self, project: Project, options: argparse.Namespace) -> None:
console = Console()
deps = [d.name for d in project.get_dependencies().values() if d.name]
deps = []
for group in project.all_dependencies.values():
for dep in group.values():
if dep.name:
deps.append(dep.name)
if not project.lockfile.exists():
console.print("No lockfile found. Please run `pdm lock` first.", style="red")
console.print(
"No lockfile found. Please run `pdm lock` first.", style="red"
)
sys.exit(1)
pinned_versions = {d["name"]: d["version"] for d in project.lockfile["package"]}
requested_version = Version(".".join(options.python_version.split(".")[:2]))
Expand Down Expand Up @@ -115,7 +121,7 @@ def handle(self, project: Project, options: argparse.Namespace) -> None:
)
for dep, pinned_version, _, supported_versions in supported:
console.print(
f" [bold green]✓[/bold green] {dep} ({latest_version})"
f" [bold green]✓[/bold green] {dep} ({pinned_version})"
)
if needs_update:
console.print(f"[bold]Update required ({len(needs_update)}):[/bold]")
Expand Down
8 changes: 5 additions & 3 deletions tests/test_readiness.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,11 @@
@pytest.fixture
def project(tmp_path):
project = mock.MagicMock(spec=Project)
project.get_dependencies.return_value = {
"foo": Requirement("foo"),
"bar": Requirement("bar"),
project.all_dependencies = {
"default": {
"foo": Requirement("foo"),
"bar": Requirement("bar"),
}
}
project.lockfile.exists.return_value = True
project.lockfile.__getitem__.side_effect = lambda _: [
Expand Down

0 comments on commit 24f15a9

Please sign in to comment.