Skip to content

Commit

Permalink
chore(packages): remove type errors
Browse files Browse the repository at this point in the history
  • Loading branch information
branchvincent authored and neersighted committed Mar 2, 2022
1 parent d158d1b commit 16c2610
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 8 deletions.
2 changes: 0 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,6 @@ files = "src"

[[tool.mypy.overrides]]
module = [
'poetry.config.file_config_source',
'poetry.console.application',
'poetry.console.logging.formatters.builder_formatter',
'poetry.console.commands.init',
Expand All @@ -129,7 +128,6 @@ module = [
'poetry.mixology.solutions.solutions.python_requirement_solution',
'poetry.mixology.term',
'poetry.mixology.version_solver',
'poetry.packages.locker',
'poetry.repositories.installed_repository',
'poetry.utils.appdirs',
'poetry.utils.authenticator',
Expand Down
13 changes: 7 additions & 6 deletions src/poetry/packages/locker.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@


if TYPE_CHECKING:
from tomlkit.items import InlineTable
from tomlkit.toml_document import TOMLDocument

from poetry.repositories import Repository
Expand Down Expand Up @@ -147,6 +148,8 @@ def locked_repository(self, with_dev_reqs: bool = False) -> Repository:
except InvalidRequirement:
# handle lock files with invalid PEP 508
m = re.match(r"^(.+?)(?:\[(.+?)])?(?:\s+\((.+)\))?$", dep)
if not m:
raise
dep_name = m.group(1)
extras = m.group(2) or ""
constraint = m.group(3) or "*"
Expand Down Expand Up @@ -289,7 +292,7 @@ def get_project_dependencies(
) -> Iterable[Dependency]:
# group packages entries by name, this is required because requirement might use
# different constraints
packages_by_name = {}
packages_by_name: dict[str, list[Package]] = {}
for pkg in locked_packages:
if pkg.name not in packages_by_name:
packages_by_name[pkg.name] = []
Expand Down Expand Up @@ -349,11 +352,9 @@ def get_project_dependency_packages(
repository = self.locked_repository(with_dev_reqs=dev)

# Build a set of all packages required by our selected extras
extra_package_names = (
None if (isinstance(extras, bool) and extras is True) else ()
)
extra_package_names: set[str] | None = None

if extra_package_names is not None:
if extras is not True:
extra_package_names = set(
get_extra_package_names(
repository.packages,
Expand Down Expand Up @@ -505,7 +506,7 @@ def _lock_packages(self, packages: list[Package]) -> list:
return locked

def _dump_package(self, package: Package) -> dict:
dependencies = {}
dependencies: dict[str, list[InlineTable]] = {}
for dependency in sorted(package.requires, key=lambda d: d.name):
if dependency.pretty_name not in dependencies:
dependencies[dependency.pretty_name] = []
Expand Down

0 comments on commit 16c2610

Please sign in to comment.