Skip to content
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

Update flake8 to 3.7.6 #6282

Merged
merged 6 commits into from
Feb 22, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ exclude =
_vendor,
data
select = E,W,F
ignore = W504

[mypy]
follow_imports = silent
Expand Down
4 changes: 2 additions & 2 deletions src/pip/_internal/build_env.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@
from pip._internal.utils.ui import open_spinner

if MYPY_CHECK_RUNNING:
from typing import Tuple, Set, Iterable, Optional, List # noqa: F401
from pip._internal.index import PackageFinder # noqa: F401
from typing import Tuple, Set, Iterable, Optional, List
from pip._internal.index import PackageFinder

logger = logging.getLogger(__name__)

Expand Down
4 changes: 2 additions & 2 deletions src/pip/_internal/cache.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
from pip._internal.wheel import InvalidWheelFilename, Wheel

if MYPY_CHECK_RUNNING:
from typing import Optional, Set, List, Any # noqa: F401
from pip._internal.index import FormatControl # noqa: F401
from typing import Optional, Set, List, Any
from pip._internal.index import FormatControl

logger = logging.getLogger(__name__)

Expand Down
8 changes: 4 additions & 4 deletions src/pip/_internal/cli/base_command.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,10 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Optional, List, Tuple, Any # noqa: F401
from optparse import Values # noqa: F401
from pip._internal.cache import WheelCache # noqa: F401
from pip._internal.req.req_set import RequirementSet # noqa: F401
from typing import Optional, List, Tuple, Any
from optparse import Values
from pip._internal.cache import WheelCache
from pip._internal.req.req_set import RequirementSet

__all__ = ['Command']

Expand Down
6 changes: 3 additions & 3 deletions src/pip/_internal/cli/cmdoptions.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@
from pip._internal.utils.ui import BAR_TYPES

if MYPY_CHECK_RUNNING:
from typing import Any, Callable, Dict, List, Optional, Union # noqa: F401
from optparse import OptionParser, Values # noqa: F401
from pip._internal.cli.parser import ConfigOptionParser # noqa: F401
from typing import Any, Callable, Dict, Optional
from optparse import OptionParser, Values
from pip._internal.cli.parser import ConfigOptionParser


def raise_option_error(parser, option, msg):
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/cli/main_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Tuple, List # noqa: F401
from typing import Tuple, List


__all__ = ["create_main_parser", "parse_command"]
Expand Down
4 changes: 2 additions & 2 deletions src/pip/_internal/commands/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import List, Type # noqa: F401
from pip._internal.cli.base_command import Command # noqa: F401
from typing import List, Type
from pip._internal.cli.base_command import Command

commands_order = [
InstallCommand,
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/configuration.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
from typing import (
Any, Dict, Iterable, List, NewType, Optional, Tuple
)

Expand Down
8 changes: 4 additions & 4 deletions src/pip/_internal/download.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,12 +48,12 @@
from pip._internal.vcs import vcs

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
from typing import (
Optional, Tuple, Dict, IO, Text, Union
)
from pip._internal.models.link import Link # noqa: F401
from pip._internal.utils.hashes import Hashes # noqa: F401
from pip._internal.vcs import AuthInfo # noqa: F401
from pip._internal.models.link import Link
from pip._internal.utils.hashes import Hashes
from pip._internal.vcs import AuthInfo

try:
import ssl # noqa
Expand Down
4 changes: 2 additions & 2 deletions src/pip/_internal/exceptions.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Optional # noqa: F401
from pip._internal.req.req_install import InstallRequirement # noqa: F401
from typing import Optional
from pip._internal.req.req_install import InstallRequirement


class PipError(Exception):
Expand Down
14 changes: 7 additions & 7 deletions src/pip/_internal/index.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,15 +41,15 @@
from pip._internal.wheel import Wheel

if MYPY_CHECK_RUNNING:
from logging import Logger # noqa: F401
from typing import ( # noqa: F401
from logging import Logger
from typing import (
Tuple, Optional, Any, List, Union, Callable, Set, Sequence,
Iterable, MutableMapping
)
from pip._vendor.packaging.version import _BaseVersion # noqa: F401
from pip._vendor.requests import Response # noqa: F401
from pip._internal.req import InstallRequirement # noqa: F401
from pip._internal.download import PipSession # noqa: F401
from pip._vendor.packaging.version import _BaseVersion
from pip._vendor.requests import Response
from pip._internal.req import InstallRequirement
from pip._internal.download import PipSession

SecureOrigin = Tuple[str, str, Optional[str]]
BuildTag = Tuple[Any, ...] # either emply tuple or Tuple[int, str]
Expand Down Expand Up @@ -227,7 +227,7 @@ def _get_html_page(link, session=None):

try:
resp = _get_html_response(url, session=session)
except _NotHTTP as exc:
except _NotHTTP:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you move this into a separate commit from removing #noqa?

logger.debug(
'Skipping page %s because it looks like an archive, and cannot '
'be checked by HEAD.', link,
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/locations.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Any, Union, Dict, List, Optional # noqa: F401
from typing import Any, Union, Dict, List, Optional


# Application Directories
Expand Down
6 changes: 3 additions & 3 deletions src/pip/_internal/models/candidate.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from pip._vendor.packaging.version import _BaseVersion # noqa: F401
from pip._internal.models.link import Link # noqa: F401
from typing import Any, Union # noqa: F401
from pip._vendor.packaging.version import _BaseVersion
from pip._internal.models.link import Link
from typing import Any


class InstallationCandidate(KeyBasedCompareMixin):
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/models/format_control.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Optional, Set, FrozenSet # noqa: F401
from typing import Optional, Set, FrozenSet


class FormatControl(object):
Expand Down
4 changes: 2 additions & 2 deletions src/pip/_internal/models/link.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Optional, Tuple, Union, Text # noqa: F401
from pip._internal.index import HTMLPage # noqa: F401
from typing import Optional, Tuple, Union
from pip._internal.index import HTMLPage


class Link(KeyBasedCompareMixin):
Expand Down
4 changes: 2 additions & 2 deletions src/pip/_internal/operations/check.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
logger = logging.getLogger(__name__)

if MYPY_CHECK_RUNNING:
from pip._internal.req.req_install import InstallRequirement # noqa: F401
from typing import ( # noqa: F401
from pip._internal.req.req_install import InstallRequirement
from typing import (
Any, Callable, Dict, Optional, Set, Tuple, List
)

Expand Down
6 changes: 3 additions & 3 deletions src/pip/_internal/operations/freeze.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
from typing import (
Iterator, Optional, List, Container, Set, Dict, Tuple, Iterable, Union
)
from pip._internal.cache import WheelCache # noqa: F401
from pip._vendor.pkg_resources import ( # noqa: F401
from pip._internal.cache import WheelCache
from pip._vendor.pkg_resources import (
Distribution, Requirement
)

Expand Down
10 changes: 5 additions & 5 deletions src/pip/_internal/operations/prepare.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,11 @@
from pip._internal.vcs import vcs

if MYPY_CHECK_RUNNING:
from typing import Any, Optional # noqa: F401
from pip._internal.req.req_install import InstallRequirement # noqa: F401
from pip._internal.index import PackageFinder # noqa: F401
from pip._internal.download import PipSession # noqa: F401
from pip._internal.req.req_tracker import RequirementTracker # noqa: F401
from typing import Any, Optional
from pip._internal.req.req_install import InstallRequirement
from pip._internal.index import PackageFinder
from pip._internal.download import PipSession
from pip._internal.req.req_tracker import RequirementTracker

logger = logging.getLogger(__name__)

Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/pep425tags.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
from typing import (
Tuple, Callable, List, Optional, Union, Dict
)

Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/pyproject.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Any, Tuple, Optional, List # noqa: F401
from typing import Any, Tuple, Optional, List


def _is_list_of_str(obj):
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/req/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import List, Sequence # noqa: F401
from typing import List, Sequence

__all__ = [
"RequirementSet", "InstallRequirement",
Expand Down
6 changes: 3 additions & 3 deletions src/pip/_internal/req/constructors.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,10 @@
from pip._internal.wheel import Wheel

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
Optional, Tuple, Set, Any, Union, Text, Dict,
from typing import (
Optional, Tuple, Set, Any, Union, Dict,
)
from pip._internal.cache import WheelCache # noqa: F401
from pip._internal.cache import WheelCache


__all__ = [
Expand Down
10 changes: 5 additions & 5 deletions src/pip/_internal/req/req_file.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,13 +22,13 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
from typing import (
Iterator, Tuple, Optional, List, Callable, Text
)
from pip._internal.req import InstallRequirement # noqa: F401
from pip._internal.cache import WheelCache # noqa: F401
from pip._internal.index import PackageFinder # noqa: F401
from pip._internal.download import PipSession # noqa: F401
from pip._internal.req import InstallRequirement
from pip._internal.cache import WheelCache
from pip._internal.index import PackageFinder
from pip._internal.download import PipSession

ReqFileLines = Iterator[Tuple[int, Text]]

Expand Down
16 changes: 8 additions & 8 deletions src/pip/_internal/req/req_install.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,15 +41,15 @@
from pip._internal.wheel import move_wheel_files

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
Optional, Iterable, List, Union, Any, Text, Sequence, Dict
from typing import (
Optional, Iterable, List, Union, Any, Sequence, Dict
)
from pip._internal.build_env import BuildEnvironment # noqa: F401
from pip._internal.cache import WheelCache # noqa: F401
from pip._internal.index import PackageFinder # noqa: F401
from pip._vendor.pkg_resources import Distribution # noqa: F401
from pip._vendor.packaging.specifiers import SpecifierSet # noqa: F401
from pip._vendor.packaging.markers import Marker # noqa: F401
from pip._internal.build_env import BuildEnvironment
from pip._internal.cache import WheelCache
from pip._internal.index import PackageFinder
from pip._vendor.pkg_resources import Distribution
from pip._vendor.packaging.specifiers import SpecifierSet
from pip._vendor.packaging.markers import Marker


logger = logging.getLogger(__name__)
Expand Down
4 changes: 2 additions & 2 deletions src/pip/_internal/req/req_set.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@
from pip._internal.wheel import Wheel

if MYPY_CHECK_RUNNING:
from typing import Optional, List, Tuple, Dict, Iterable # noqa: F401
from pip._internal.req.req_install import InstallRequirement # noqa: F401
from typing import Optional, List, Tuple, Dict, Iterable
from pip._internal.req.req_install import InstallRequirement


logger = logging.getLogger(__name__)
Expand Down
6 changes: 3 additions & 3 deletions src/pip/_internal/req/req_tracker.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Set, Iterator # noqa: F401
from pip._internal.req.req_install import InstallRequirement # noqa: F401
from pip._internal.models.link import Link # noqa: F401
from typing import Set, Iterator
from pip._internal.req.req_install import InstallRequirement
from pip._internal.models.link import Link

logger = logging.getLogger(__name__)

Expand Down
14 changes: 7 additions & 7 deletions src/pip/_internal/resolve.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,15 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Optional, DefaultDict, List, Set # noqa: F401
from pip._internal.download import PipSession # noqa: F401
from pip._internal.req.req_install import InstallRequirement # noqa: F401
from pip._internal.index import PackageFinder # noqa: F401
from pip._internal.req.req_set import RequirementSet # noqa: F401
from pip._internal.operations.prepare import ( # noqa: F401
from typing import Optional, DefaultDict, List, Set
from pip._internal.download import PipSession
from pip._internal.req.req_install import InstallRequirement
from pip._internal.index import PackageFinder
from pip._internal.req.req_set import RequirementSet
from pip._internal.operations.prepare import (
DistAbstraction, RequirementPreparer
)
from pip._internal.cache import WheelCache # noqa: F401
from pip._internal.cache import WheelCache

logger = logging.getLogger(__name__)

Expand Down
4 changes: 1 addition & 3 deletions src/pip/_internal/utils/appdirs.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import ( # noqa: F401
List, Union
)
from typing import List
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd be fine with keeping Union here in this commit, and removing it the next commit.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a few places other places too which remove not-used imports, it'd be nice if all of those changes could be moved into a separate commit.



def user_cache_dir(appname):
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/utils/compat.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Tuple, Text # noqa: F401
from typing import Tuple, Text

try:
import ipaddress
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/utils/deprecation.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import Any, Optional # noqa: F401
from typing import Any, Optional


class PipDeprecationWarning(Warning):
Expand Down
2 changes: 1 addition & 1 deletion src/pip/_internal/utils/encoding.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
from pip._internal.utils.typing import MYPY_CHECK_RUNNING

if MYPY_CHECK_RUNNING:
from typing import List, Tuple, Text # noqa: F401
from typing import List, Tuple, Text

BOMS = [
(codecs.BOM_UTF8, 'utf8'),
Expand Down
Loading