git_lines() - Code Metrics - Inspection of "Merge pull request #159 from michaeljoseph/refacto..." - michaeljoseph/changes - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b78565...f19f1e )

by Michael

created 2018-02-13 12:55 UTC

git_lines() A

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
c	1
b	0
f	0
dl	0
loc	2
rs	10

import re
import shlex

import attr
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import giturlparse
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import semantic_version
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from cached_property import cached_property
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from plumbum.cmd import git as git_command
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

from changes import services

from changes.compat import IS_WINDOWS

GITHUB_MERGED_PULL_REQUEST = re.compile(
    r'^([0-9a-f]{5,40}) Merge pull request #(\w+)'
)


def git(command):
    command = shlex.split(
        command,
        posix=not IS_WINDOWS
    )
    return git_command[command]()


def git_lines(command):
    return git(command).splitlines()


@attr.s
class GitRepository(object):
    VERSION_ZERO = semantic_version.Version('0.0.0')
    # TODO: handle multiple remotes (for non-owner maintainer workflows)
    REMOTE_NAME = 'origin'

    auth_token = attr.ib(default=None)

    @property
    def remote_url(self):
        return git('config --get remote.{}.url'.format(
            self.REMOTE_NAME
        ))

    @property
    def parsed_repo(self):
        return giturlparse.parse(self.remote_url)

    @property
    def repo(self):
        return self.parsed_repo.repo

    @property
    def owner(self):
        return self.parsed_repo.owner

    @property
    def platform(self):
        return self.parsed_repo.platform

    @property
    def is_github(self):
        return self.parsed_repo.github

    @property
    def is_bitbucket(self):
        return self.parsed_repo.bitbucket

    @property
    def commit_history(self):
        return [
            commit_message
            for commit_message in git_lines(
                'log --oneline --no-color'
            )
            if commit_message
        ]

    @property
    def first_commit_sha(self):
        return git('rev-list --max-parents=0 HEAD')

    @property
    def tags(self):
        return git_lines('tag --list')

    @property
    def versions(self):
        versions = []
        for tag in self.tags:
            try:
                versions.append(semantic_version.Version(tag))
            except ValueError:
                pass
        return versions

    @property
    def latest_version(self) -> semantic_version.Version:
        return max(self.versions) if self.versions else self.VERSION_ZERO

    def merges_since(self, version=None):
        if version == semantic_version.Version('0.0.0'):
            version = self.first_commit_sha

        revision_range = ' {}..HEAD'.format(version) if version else ''

        merge_commits = git(
            'log --oneline --merges --no-color{}'.format(revision_range)
        ).split('\n')
        return merge_commits

    @property
    def merges_since_latest_version(self):
        return self.merges_since(self.latest_version)

    @property
    def files_modified_in_last_commit(self):
        return git('diff --name -only --diff -filter=d')

    @property
    def dirty_files(self):
        return [
            modified_path
            for modified_path in git('-c color.status=false status --short --branch')

            if modified_path.startswith(' M')
        ]

    @staticmethod
    def add(files_to_add):
        return git('add {}'.format(' '.join(files_to_add)))

    @staticmethod
    def commit(message):
        # FIXME: message is one token
        return git_command[
            'commit',
            '--message="{}"'.format(message)
        ]()

    @staticmethod
    def discard(file_paths):
        return git('checkout -- {}'.format(' '.join(file_paths)))

    @staticmethod
    def tag(version):
        # TODO: signed tags
        return git(
            'tag --annotate {version} --message="{version}"'.format(
                version=version
            )
        )

    @staticmethod
    def push():
        return git('push --tags')


@attr.s
class GitHubRepository(GitRepository):
    api = attr.ib(default=None)

    def __attrs_post_init__(self):
        self.api = services.GitHub(self)

    @cached_property
    def labels(self):
        return self.api.labels()

    @cached_property
    def pull_requests_since_latest_version(self):

        return [
            PullRequest.from_github(self.api.pull_request(pull_request_number))
            for pull_request_number in self.pull_request_numbers_since_latest_version

        ]

    @property
    def pull_request_numbers_since_latest_version(self):

        pull_request_numbers = []

        for commit_msg in self.merges_since(self.latest_version):

            matches = GITHUB_MERGED_PULL_REQUEST.findall(commit_msg)

            if matches:
                _, pull_request_number = matches[0]
                pull_request_numbers.append(pull_request_number)

        return pull_request_numbers

    def create_release(self, release):
        return self.api.create_release(release)


@attr.s
class PullRequest(object):
    number = attr.ib()
    title = attr.ib()
    description = attr.ib()
    author = attr.ib()
    body = attr.ib()
    user = attr.ib()
    labels = attr.ib(default=attr.Factory(list))

    @property
    def description(self):
        return self.body

    @property
    def author(self):
        return self.user['login']

    @property
    def label_names(self):
        return [
            label['name']
            for label in self.labels
        ]

    @classmethod
    def from_github(cls, api_response):
        return cls(**{
            k.name: api_response[k.name]
            for k in attr.fields(cls)
        })

    @classmethod
    def from_number(cls, number):
        pass


Push — master ( b78565...f19f1e )

git_lines() A

Complexity

Size

Duplication

Importance

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. Missing Dependencies

2. Missing init.py files

1			import re
2			import shlex
3
4			import attr
			0 ignored issues – show Configuration introduced 2018-02-13 11:19 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...
5			import giturlparse
			0 ignored issues – show Configuration introduced 2018-02-13 11:19 UTC by Report Bug Copy Issue Report The import `giturlparse` 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...
6			import semantic_version
			0 ignored issues – show Configuration introduced 2018-02-13 11:19 UTC by Report Bug Copy Issue Report The import `semantic_version` 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...
7			from cached_property import cached_property
			0 ignored issues – show Configuration introduced 2017-11-12 12:09 UTC by Report Bug Copy Issue Report The import `cached_property` 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...
8			from plumbum.cmd import git as git_command
			0 ignored issues – show Configuration introduced 2018-02-13 10:29 UTC by Report Bug Copy Issue Report The import `plumbum.cmd` 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...
9
10			from changes import services
			0 ignored issues – show Bug introduced 2018-02-13 11:19 UTC by Report Bug Copy Issue Report The name `services` does not seem to exist in module `changes`. Loading history...
11			from changes.compat import IS_WINDOWS
12
13			GITHUB_MERGED_PULL_REQUEST = re.compile(
14			r'^([0-9a-f]{5,40}) Merge pull request #(\w+)'
15			)
16
17
18			def git(command):
19			command = shlex.split(
20			command,
21			posix=not IS_WINDOWS
22			)
23			return git_command[command]()
24
25
26			def git_lines(command):
27			return git(command).splitlines()
28
29
30			@attr.s
31			class GitRepository(object):
32			VERSION_ZERO = semantic_version.Version('0.0.0')
33			# TODO: handle multiple remotes (for non-owner maintainer workflows)
34			REMOTE_NAME = 'origin'
35
36			auth_token = attr.ib(default=None)
37
38			@property
39			def remote_url(self):
40			return git('config --get remote.{}.url'.format(
41			self.REMOTE_NAME
42			))
43
44			@property
45			def parsed_repo(self):
46			return giturlparse.parse(self.remote_url)
47
48			@property
49			def repo(self):
50			return self.parsed_repo.repo
51
52			@property
53			def owner(self):
54			return self.parsed_repo.owner
55
56			@property
57			def platform(self):
58			return self.parsed_repo.platform
59
60			@property
61			def is_github(self):
62			return self.parsed_repo.github
63
64			@property
65			def is_bitbucket(self):
66			return self.parsed_repo.bitbucket
67
68			@property
69			def commit_history(self):
70			return [
71			commit_message
72			for commit_message in git_lines(
73			'log --oneline --no-color'
74			)
75			if commit_message
76			]
77
78			@property
79			def first_commit_sha(self):
80			return git('rev-list --max-parents=0 HEAD')
81
82			@property
83			def tags(self):
84			return git_lines('tag --list')
85
86			@property
87			def versions(self):
88			versions = []
89			for tag in self.tags:
90			try:
91			versions.append(semantic_version.Version(tag))
92			except ValueError:
93			pass
94			return versions
95
96			@property
97			def latest_version(self) -> semantic_version.Version:
98			return max(self.versions) if self.versions else self.VERSION_ZERO
99
100			def merges_since(self, version=None):
101			if version == semantic_version.Version('0.0.0'):
102			version = self.first_commit_sha
103
104			revision_range = ' {}..HEAD'.format(version) if version else ''
105
106			merge_commits = git(
107			'log --oneline --merges --no-color{}'.format(revision_range)
108			).split('\n')
109			return merge_commits
110
111			@property
112			def merges_since_latest_version(self):
113			return self.merges_since(self.latest_version)
114
115			@property
116			def files_modified_in_last_commit(self):
117			return git('diff --name -only --diff -filter=d')
118
119			@property
120			def dirty_files(self):
121			return [
122			modified_path
123			for modified_path in git('-c color.status=false status --short --branch')
			0 ignored issues – show Coding Style introduced 2018-02-13 10:29 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...
124			if modified_path.startswith(' M')
125			]
126
127			@staticmethod
128			def add(files_to_add):
129			return git('add {}'.format(' '.join(files_to_add)))
130
131			@staticmethod
132			def commit(message):
133			# FIXME: message is one token
134			return git_command[
135			'commit',
136			'--message="{}"'.format(message)
137			]()
138
139			@staticmethod
140			def discard(file_paths):
141			return git('checkout -- {}'.format(' '.join(file_paths)))
142
143			@staticmethod
144			def tag(version):
145			# TODO: signed tags
146			return git(
147			'tag --annotate {version} --message="{version}"'.format(
148			version=version
149			)
150			)
151
152			@staticmethod
153			def push():
154			return git('push --tags')
155
156
157			@attr.s
158			class GitHubRepository(GitRepository):
159			api = attr.ib(default=None)
160
161			def __attrs_post_init__(self):
162			self.api = services.GitHub(self)
163
164			@cached_property
165			def labels(self):
166			return self.api.labels()
167
168			@cached_property
169			def pull_requests_since_latest_version(self):
			0 ignored issues – show Coding Style Naming introduced 2017-11-12 12:09 UTC by Report Bug Copy Issue Report The name `pull_requests_since_latest_version` does not conform to the method 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...
170			return [
171			PullRequest.from_github(self.api.pull_request(pull_request_number))
172			for pull_request_number in self.pull_request_numbers_since_latest_version
			0 ignored issues – show Coding Style introduced 2017-11-02 21:46 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...
173			]
174
175			@property
176			def pull_request_numbers_since_latest_version(self):
			0 ignored issues – show Coding Style Naming introduced 2017-11-02 21:46 UTC by Report Bug Copy Issue Report The name `pull_request_numbers_since_latest_version` does not conform to the attribute 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...
177			pull_request_numbers = []
178
179			for commit_msg in self.merges_since(self.latest_version):
180
181			matches = GITHUB_MERGED_PULL_REQUEST.findall(commit_msg)
182
183			if matches:
184			_, pull_request_number = matches[0]
185			pull_request_numbers.append(pull_request_number)
186
187			return pull_request_numbers
188
189			def create_release(self, release):
190			return self.api.create_release(release)
191
192
193			@attr.s
194			class PullRequest(object):
195			number = attr.ib()
196			title = attr.ib()
197			description = attr.ib()
198			author = attr.ib()
199			body = attr.ib()
200			user = attr.ib()
201			labels = attr.ib(default=attr.Factory(list))
202
203			@property
204			def description(self):
205			return self.body
206
207			@property
208			def author(self):
209			return self.user['login']
210
211			@property
212			def label_names(self):
213			return [
214			label['name']
215			for label in self.labels
216			]
217
218			@classmethod
219			def from_github(cls, api_response):
220			return cls(**{
221			k.name: api_response[k.name]
222			for k in attr.fields(cls)
223			})
224
225			@classmethod
226			def from_number(cls, number):
227			pass
228

michaeljoseph / changes

Push — master ( b78565...f19f1e )

git_lines() A

Complexity

Size

Duplication

Importance

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. 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

2. Missing init.py files