choose_labels() - Code Metrics - Inspection of "Simplify, since there is no longer an init cli sub..." - michaeljoseph/changes - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — publish ( fee4e2...70e944 )

by Michael

created 2017-10-26 21:24 UTC

choose_labels() D

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	11
c	0
b	0
f	0
dl	0
loc	56
rs	4.4262

How to fix Long Method Complexity

import re
import textwrap
from collections import OrderedDict
from configparser import RawConfigParser
from os.path import exists, expanduser, expandvars, join, curdir
import io
import os
import sys

import click
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from pathlib import Path

import inflect
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import toml
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import attr
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

import changes
from changes.models import GitRepository
from .commands import info, note, debug, error

AUTH_TOKEN_ENVVAR = 'GITHUB_AUTH_TOKEN'

# via https://github.com/jakubroztocil/httpie/blob/6bdfc7a/httpie/config.py#L9
IS_WINDOWS = 'win32' in str(sys.platform).lower()
DEFAULT_CONFIG_FILE = str(os.environ.get(
    'CHANGES_CONFIG_FILE',
    expanduser('~/.changes') if not IS_WINDOWS else
    expandvars(r'%APPDATA%\\.changes')
))

PROJECT_CONFIG_FILE = '.changes.toml'
DEFAULT_RELEASES_DIRECTORY = 'docs/releases'


@attr.s
class Changes(object):
    auth_token = attr.ib()

    @classmethod
    def load(cls):
        tool_config_path = Path(str(os.environ.get(
            'CHANGES_CONFIG_FILE',
            expanduser('~/.changes') if not IS_WINDOWS else
            expandvars(r'%APPDATA%\\.changes')
        )))

        tool_settings = None
        if tool_config_path.exists():
            tool_settings = Changes(
                **(toml.load(tool_config_path.open())['changes'])
            )

        # envvar takes precedence over config file settings
        auth_token = os.environ.get(AUTH_TOKEN_ENVVAR)
        if auth_token:
            info('Found Github Auth Token in the environment')
            tool_settings = Changes(auth_token=auth_token)
        elif not (tool_settings and tool_settings.auth_token):
            while not auth_token:
                info('No auth token found, asking for it')
                # to interact with the Git*H*ub API
                note('You need a Github Auth Token for changes to create a release.')

                click.pause('Press [enter] to launch the GitHub "New personal access '

                            'token" page, to create a token for changes.')
                click.launch('https://github.com/settings/tokens/new')
                auth_token = click.prompt('Enter your changes token')

            if not tool_settings:
                tool_settings = Changes(auth_token=auth_token)

            tool_config_path.write_text(

                toml.dumps({
                    'changes': attr.asdict(tool_settings)
                })
            )

        return tool_settings


@attr.s
class Project(object):
    releases_directory = attr.ib()
    repository = attr.ib(default=None)
    bumpversion = attr.ib(default=None)
    labels = attr.ib(default=attr.Factory(dict))

    @classmethod
    def load(cls):
        repository = GitRepository(
            auth_token=changes.settings.auth_token
        )

        project_settings = configure_changes(repository)
        project_settings.repository = repository
        project_settings.bumpversion = BumpVersion.load(repository.latest_version)


        return project_settings


@attr.s
class BumpVersion(object):
    DRAFT_OPTIONS = [
        '--dry-run', '--verbose',
        '--no-commit', '--no-tag',
        '--allow-dirty',
    ]
    STAGE_OPTIONS = [
        '--verbose', '--allow-dirty',
        '--no-commit', '--no-tag',
    ]

    current_version = attr.ib()
    version_files_to_replace = attr.ib(default=attr.Factory(list))

    @classmethod
    def load(cls, latest_version):
        return configure_bumpversion(latest_version)

    @classmethod
    def read_from_file(cls, config_path: Path):
        config = RawConfigParser('')
        config.readfp(config_path.open('rt', encoding='utf-8'))

        current_version = config.get("bumpversion", 'current_version')

        filenames = []
        for section_name in config.sections():

            section_name_match = re.compile("^bumpversion:(file|part):(.+)").match(section_name)


            if not section_name_match:
                continue

            section_prefix, section_value = section_name_match.groups()

            if section_prefix == "file":
                filenames.append(section_value)

        return cls(
            current_version=current_version,
            version_files_to_replace=filenames,
        )

    def write_to_file(self, config_path: Path):
        bumpversion_cfg = textwrap.dedent(
            """\
            [bumpversion]
            current_version = {current_version}

            """
        ).format(**attr.asdict(self))

        bumpversion_files = '\n\n'.join([
            '[bumpversion:file:{}]'.format(file_name)
            for file_name in self.version_files_to_replace
        ])

        config_path.write_text(
            bumpversion_cfg + bumpversion_files
        )


def configure_changes(repository):
    changes_project_config_path = Path(PROJECT_CONFIG_FILE)
    project_settings = None
    if changes_project_config_path.exists():
        # releases_directory, labels
        project_settings = Project(
            **(toml.load(changes_project_config_path.open())['changes'])
        )

    if not project_settings:
        releases_directory = Path(click.prompt(
            'Enter the directory to store your releases notes',
            DEFAULT_RELEASES_DIRECTORY,
            type=click.Path(exists=True, dir_okay=True)
        ))

        if not releases_directory.exists():
            debug('Releases directory {} not found, creating it.'.format(releases_directory))

            releases_directory.mkdir(parents=True)

        # FIXME: GitHub(repository).labels()
        project_settings = Project(
            releases_directory=str(releases_directory),
            labels=configure_labels(repository.github_labels()),
        )
        # write config file
        changes_project_config_path.write_text(

            toml.dumps({
                'changes': attr.asdict(project_settings)
            })
        )

    return project_settings


def configure_bumpversion(latest_version):
    # TODO: look in other supported bumpversion config locations
    bumpversion = None
    bumpversion_config_path = Path('.bumpversion.cfg')
    if not bumpversion_config_path.exists():
        user_supplied_versioned_file_paths = []


        version_file_path_answer = None
        input_terminator = '.'
        while not version_file_path_answer == input_terminator:
            version_file_path_answer = click.prompt(
                'Enter a path to a file that contains a version number '
                "(enter a path of '.' when you're done selecting files)",
                type=click.Path(
                    exists=True,
                    dir_okay=True,
                    file_okay=True,
                    readable=True
                )
            )

            if version_file_path_answer != input_terminator:
                user_supplied_versioned_file_paths.append(version_file_path_answer)


        bumpversion = BumpVersion(
            current_version=latest_version,
            version_files_to_replace=user_supplied_versioned_file_paths,
        )
        bumpversion.write_to_file(bumpversion_config_path)

    return bumpversion


def configure_labels(github_labels):
    labels_keyed_by_name = {}
    for label in github_labels:
        labels_keyed_by_name[label['name']] = label

    # TODO: streamlined support for github defaults: enhancement, bug
    changelog_worthy_labels = choose_labels([
        properties['name']
        for _, properties in labels_keyed_by_name.items()
    ])

    # TODO: apply description transform in labels_prompt function
    described_labels = {}
    # auto-generate label descriptions
    for label_name in changelog_worthy_labels:
        label_properties = labels_keyed_by_name[label_name]
        # Auto-generate description as pluralised titlecase label name
        label_properties['description'] = inflect.engine().plural(label_name.title())


        described_labels[label_name] = label_properties

    return described_labels


def choose_labels(alternatives):
    """
    Prompt the user select several labels from the provided alternatives.

    At least one label must be selected

    :param list alternatives: Sequence of options that are available to select from

    :return: Several selected labels
    """
    if not alternatives:
        raise ValueError

    if not isinstance(alternatives, list):
        raise TypeError

    choice_map = OrderedDict(
      ('{}'.format(i), value) for i, value in enumerate(alternatives, 1)
    )
    # prepend a termination option
    input_terminator = '0'
    choice_map.update({input_terminator: '<done>'})
    choice_map.move_to_end('0', last=False)

    choice_indexes = choice_map.keys()

    choice_lines = ['{} - {}'.format(*c) for c in choice_map.items()]
    prompt = '\n'.join((
        'Select labels:',
        '\n'.join(choice_lines),
        'Choose from {}'.format(', '.join(choice_indexes))
    ))

    user_choices = set()
    user_choice = None

    while not user_choice == input_terminator:
        if user_choices:
            note('Selected labels: [{}]'.format(','.join(user_choices)))

        user_choice = click.prompt(
            prompt,
            type=click.Choice(choice_indexes),
            default=input_terminator,
        )
        done = user_choice == input_terminator
        new_selection = user_choice not in user_choices
        nothing_selected = not user_choices

        if not done and new_selection:
            user_choices.add(choice_map[user_choice])

        if done and nothing_selected:
            error('Please select at least one label')
            user_choice = None

    return user_choices

DEFAULTS = {
    'changelog': 'CHANGELOG.md',
    'readme': 'README.md',
    'github_auth_token': None,
}


class Config:
    test_command = None
    pypi = None
    skip_changelog = None
    changelog_content = None
    repo = None

    def __init__(self, module_name, dry_run, debug, no_input, requirements,
param = 5

class Foo:
    def __init__(self, param):   # "param" would be flagged here
        self.param = param
                 new_version, current_version, repo_url, version_prefix):
        self.module_name = module_name
        # module_name => project_name => curdir
        self.dry_run = dry_run
        self.debug = debug
        self.no_input = no_input
        self.requirements = requirements
        self.new_version = (
            version_prefix + new_version
            if version_prefix
            else new_version
        )
        self.current_version = current_version


def project_config():
    """Deprecated"""
    project_name = curdir

    config_path = Path(join(project_name, PROJECT_CONFIG_FILE))

    if not exists(config_path):
        store_settings(DEFAULTS.copy())
        return DEFAULTS

    return toml.load(io.open(config_path)) or {}


def store_settings(settings):
    pass



Push — publish ( fee4e2...70e944 )

choose_labels() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Complexity

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

1			import re
2			import textwrap
3			from collections import OrderedDict
4			from configparser import RawConfigParser
5			from os.path import exists, expanduser, expandvars, join, curdir
6			import io
7			import os
8			import sys
9
10			import click
			0 ignored issues – show Configuration introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report The import `click` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
11			from pathlib import Path
12
13			import inflect
			0 ignored issues – show Configuration introduced 2017-10-26 21:30 UTC by Report Bug Copy Issue Report The import `inflect` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
14			import toml
			0 ignored issues – show Configuration introduced 2017-10-26 21:30 UTC by Report Bug Copy Issue Report The import `toml` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
15			import attr
			0 ignored issues – show Configuration introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report The import `attr` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
16
17			import changes
18			from changes.models import GitRepository
19			from .commands import info, note, debug, error
20
21			AUTH_TOKEN_ENVVAR = 'GITHUB_AUTH_TOKEN'
22
23			# via https://github.com/jakubroztocil/httpie/blob/6bdfc7a/httpie/config.py#L9
24			IS_WINDOWS = 'win32' in str(sys.platform).lower()
25			DEFAULT_CONFIG_FILE = str(os.environ.get(
26			'CHANGES_CONFIG_FILE',
27			expanduser('~/.changes') if not IS_WINDOWS else
28			expandvars(r'%APPDATA%\\.changes')
29			))
30
31			PROJECT_CONFIG_FILE = '.changes.toml'
32			DEFAULT_RELEASES_DIRECTORY = 'docs/releases'
33
34
35			@attr.s
36			class Changes(object):
37			auth_token = attr.ib()
38
39			@classmethod
40			def load(cls):
41			tool_config_path = Path(str(os.environ.get(
42			'CHANGES_CONFIG_FILE',
43			expanduser('~/.changes') if not IS_WINDOWS else
44			expandvars(r'%APPDATA%\\.changes')
45			)))
46
47			tool_settings = None
48			if tool_config_path.exists():
49			tool_settings = Changes(
50			**(toml.load(tool_config_path.open())['changes'])
51			)
52
53			# envvar takes precedence over config file settings
54			auth_token = os.environ.get(AUTH_TOKEN_ENVVAR)
55			if auth_token:
56			info('Found Github Auth Token in the environment')
57			tool_settings = Changes(auth_token=auth_token)
58			elif not (tool_settings and tool_settings.auth_token):
59			while not auth_token:
60			info('No auth token found, asking for it')
61			# to interact with the GitHub API
62			note('You need a Github Auth Token for changes to create a release.')
			0 ignored issues – show Coding Style introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (85/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
63			click.pause('Press [enter] to launch the GitHub "New personal access '
			0 ignored issues – show Coding Style introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (86/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
64			'token" page, to create a token for changes.')
65			click.launch('https://github.com/settings/tokens/new')
66			auth_token = click.prompt('Enter your changes token')
67
68			if not tool_settings:
69			tool_settings = Changes(auth_token=auth_token)
70
71			tool_config_path.write_text(
			0 ignored issues – show Bug introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report The Instance of `Path` does not seem to have a member named `write_text`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
72			toml.dumps({
73			'changes': attr.asdict(tool_settings)
74			})
75			)
76
77			return tool_settings
78
79
80			@attr.s
81			class Project(object):
82			releases_directory = attr.ib()
83			repository = attr.ib(default=None)
84			bumpversion = attr.ib(default=None)
85			labels = attr.ib(default=attr.Factory(dict))
86
87			@classmethod
88			def load(cls):
89			repository = GitRepository(
90			auth_token=changes.settings.auth_token
91			)
92
93			project_settings = configure_changes(repository)
94			project_settings.repository = repository
95			project_settings.bumpversion = BumpVersion.load(repository.latest_version)
			0 ignored issues – show Coding Style introduced 2017-10-21 21:51 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (82/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
96
97			return project_settings
98
99
100			@attr.s
101			class BumpVersion(object):
102			DRAFT_OPTIONS = [
103			'--dry-run', '--verbose',
104			'--no-commit', '--no-tag',
105			'--allow-dirty',
106			]
107			STAGE_OPTIONS = [
108			'--verbose', '--allow-dirty',
109			'--no-commit', '--no-tag',
110			]
111
112			current_version = attr.ib()
113			version_files_to_replace = attr.ib(default=attr.Factory(list))
114
115			@classmethod
116			def load(cls, latest_version):
117			return configure_bumpversion(latest_version)
118
119			@classmethod
120			def read_from_file(cls, config_path: Path):
121			config = RawConfigParser('')
122			config.readfp(config_path.open('rt', encoding='utf-8'))
123
124			current_version = config.get("bumpversion", 'current_version')
125
126			filenames = []
127			for section_name in config.sections():
128
129			section_name_match = re.compile("^bumpversion:(file\|part):(.+)").match(section_name)
			0 ignored issues – show Coding Style introduced 2017-10-26 17:35 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (96/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
130
131			if not section_name_match:
132			continue
133
134			section_prefix, section_value = section_name_match.groups()
135
136			if section_prefix == "file":
137			filenames.append(section_value)
138
139			return cls(
140			current_version=current_version,
141			version_files_to_replace=filenames,
142			)
143
144			def write_to_file(self, config_path: Path):
145			bumpversion_cfg = textwrap.dedent(
146			"""\
147			[bumpversion]
148			current_version = {current_version}
149
150			"""
151			).format(**attr.asdict(self))
152
153			bumpversion_files = '\n\n'.join([
154			'[bumpversion:file:{}]'.format(file_name)
155			for file_name in self.version_files_to_replace
156			])
157
158			config_path.write_text(
159			bumpversion_cfg + bumpversion_files
160			)
161
162
163			def configure_changes(repository):
164			changes_project_config_path = Path(PROJECT_CONFIG_FILE)
165			project_settings = None
166			if changes_project_config_path.exists():
167			# releases_directory, labels
168			project_settings = Project(
169			**(toml.load(changes_project_config_path.open())['changes'])
170			)
171
172			if not project_settings:
173			releases_directory = Path(click.prompt(
174			'Enter the directory to store your releases notes',
175			DEFAULT_RELEASES_DIRECTORY,
176			type=click.Path(exists=True, dir_okay=True)
177			))
178
179			if not releases_directory.exists():
180			debug('Releases directory {} not found, creating it.'.format(releases_directory))
			0 ignored issues – show Coding Style introduced 2017-10-24 22:09 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (93/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
181			releases_directory.mkdir(parents=True)
182
183			# FIXME: GitHub(repository).labels()
184			project_settings = Project(
185			releases_directory=str(releases_directory),
186			labels=configure_labels(repository.github_labels()),
187			)
188			# write config file
189			changes_project_config_path.write_text(
			0 ignored issues – show Bug introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report The Instance of `Path` does not seem to have a member named `write_text`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
190			toml.dumps({
191			'changes': attr.asdict(project_settings)
192			})
193			)
194
195			return project_settings
196
197
198			def configure_bumpversion(latest_version):
199			# TODO: look in other supported bumpversion config locations
200			bumpversion = None
201			bumpversion_config_path = Path('.bumpversion.cfg')
202			if not bumpversion_config_path.exists():
203			user_supplied_versioned_file_paths = []
			0 ignored issues – show Coding Style Naming introduced 2017-10-18 22:58 UTC by Report Bug Copy Issue Report The name `user_supplied_versioned_file_paths` does not conform to the variable naming conventions (`[a-z_][a-z0-9_]{2,30}$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
204
205			version_file_path_answer = None
206			input_terminator = '.'
207			while not version_file_path_answer == input_terminator:
208			version_file_path_answer = click.prompt(
209			'Enter a path to a file that contains a version number '
210			"(enter a path of '.' when you're done selecting files)",
211			type=click.Path(
212			exists=True,
213			dir_okay=True,
214			file_okay=True,
215			readable=True
216			)
217			)
218
219			if version_file_path_answer != input_terminator:
220			user_supplied_versioned_file_paths.append(version_file_path_answer)
			0 ignored issues – show Coding Style introduced 2017-10-24 22:09 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (83/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
221
222			bumpversion = BumpVersion(
223			current_version=latest_version,
224			version_files_to_replace=user_supplied_versioned_file_paths,
225			)
226			bumpversion.write_to_file(bumpversion_config_path)
227
228			return bumpversion
229
230
231			def configure_labels(github_labels):
232			labels_keyed_by_name = {}
233			for label in github_labels:
234			labels_keyed_by_name[label['name']] = label
235
236			# TODO: streamlined support for github defaults: enhancement, bug
237			changelog_worthy_labels = choose_labels([
238			properties['name']
239			for _, properties in labels_keyed_by_name.items()
240			])
241
242			# TODO: apply description transform in labels_prompt function
243			described_labels = {}
244			# auto-generate label descriptions
245			for label_name in changelog_worthy_labels:
246			label_properties = labels_keyed_by_name[label_name]
247			# Auto-generate description as pluralised titlecase label name
248			label_properties['description'] = inflect.engine().plural(label_name.title())
			0 ignored issues – show Coding Style introduced 2017-10-26 21:30 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (85/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
249
250			described_labels[label_name] = label_properties
251
252			return described_labels
253
254
255			def choose_labels(alternatives):
256			"""
257			Prompt the user select several labels from the provided alternatives.
258
259			At least one label must be selected
260
261			:param list alternatives: Sequence of options that are available to select from
			0 ignored issues – show Coding Style introduced 2017-10-26 21:30 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (83/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
262			:return: Several selected labels
263			"""
264			if not alternatives:
265			raise ValueError
266
267			if not isinstance(alternatives, list):
268			raise TypeError
269
270			choice_map = OrderedDict(
271			('{}'.format(i), value) for i, value in enumerate(alternatives, 1)
272			)
273			# prepend a termination option
274			input_terminator = '0'
275			choice_map.update({input_terminator: '<done>'})
276			choice_map.move_to_end('0', last=False)
277
278			choice_indexes = choice_map.keys()
279
280			choice_lines = ['{} - {}'.format(*c) for c in choice_map.items()]
281			prompt = '\n'.join((
282			'Select labels:',
283			'\n'.join(choice_lines),
284			'Choose from {}'.format(', '.join(choice_indexes))
285			))
286
287			user_choices = set()
288			user_choice = None
289
290			while not user_choice == input_terminator:
291			if user_choices:
292			note('Selected labels: [{}]'.format(','.join(user_choices)))
293
294			user_choice = click.prompt(
295			prompt,
296			type=click.Choice(choice_indexes),
297			default=input_terminator,
298			)
299			done = user_choice == input_terminator
300			new_selection = user_choice not in user_choices
301			nothing_selected = not user_choices
302
303			if not done and new_selection:
304			user_choices.add(choice_map[user_choice])
305
306			if done and nothing_selected:
307			error('Please select at least one label')
308			user_choice = None
309
310			return user_choices
311
312			DEFAULTS = {
313			'changelog': 'CHANGELOG.md',
314			'readme': 'README.md',
315			'github_auth_token': None,
316			}
317
318
319			class Config:
320			test_command = None
321			pypi = None
322			skip_changelog = None
323			changelog_content = None
324			repo = None
325
326			def __init__(self, module_name, dry_run, debug, no_input, requirements,
			0 ignored issues – show Comprehensibility Bug introduced 2017-10-24 22:29 UTC by Report Bug Copy Issue Report `debug` is re-defining a name which is already available in the outer-scope (previously defined on line `19`). It is generally a bad practice to shadow variables from the outer-scope. In most cases, this is done unintentionally and might lead to unexpected behavior: param = 5 class Foo: def __init__(self, param): # "param" would be flagged here self.param = param Loading history...
327			new_version, current_version, repo_url, version_prefix):
328			self.module_name = module_name
329			# module_name => project_name => curdir
330			self.dry_run = dry_run
331			self.debug = debug
332			self.no_input = no_input
333			self.requirements = requirements
334			self.new_version = (
335			version_prefix + new_version
336			if version_prefix
337			else new_version
338			)
339			self.current_version = current_version
340
341
342			def project_config():
343			"""Deprecated"""
344			project_name = curdir
345
346			config_path = Path(join(project_name, PROJECT_CONFIG_FILE))
347
348			if not exists(config_path):
349			store_settings(DEFAULTS.copy())
350			return DEFAULTS
351
352			return toml.load(io.open(config_path)) or {}
353
354
355			def store_settings(settings):
356			pass
357
358

michaeljoseph / changes

Push — publish ( fee4e2...70e944 )

choose_labels() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Complexity

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files

2. Missing init.py files

2. Missing init.py files

2. Missing init.py files