GitRepository.bitbucket() - Code Metrics - Inspection of "Merge pull request #120 from michaeljoseph/stage" - michaeljoseph/changes - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — pyup-initial-update ( 5e7443 )

by Michael

created 2017-10-23 20:25 UTC

GitRepository.bitbucket() A

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	1
CRAP Score	1.125

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
c	1
b	0
f	0
dl	0
loc	3
ccs	1
cts	2
cp	0.5
crap	1.125
rs	10

import re
import shlex

import attr
# .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
import uritemplate
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import requests
import giturlparse
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from plumbum.cmd import git
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

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

GITHUB_PULL_REQUEST_API = (
    'https://api.github.com/repos{/owner}{/repo}/issues{/number}'
)


def changes_to_release_type(repository):
    pull_request_labels = set()
    changes = repository.changes_since_last_version

    for change in changes:
        for label in change.labels:
            pull_request_labels.add(label)

    change_descriptions = [
        '\n'.join([change.title, change.description]) for change in changes
    ]

    current_version = repository.latest_version
    if 'BREAKING CHANGE' in change_descriptions:
        return 'major', Release.BREAKING_CHANGE, current_version.next_major()
    elif 'enhancement' in pull_request_labels:
        return 'minor', Release.FEATURE, current_version.next_minor()
    elif 'bug' in pull_request_labels:
        return 'patch', Release.FIX, current_version.next_patch()
    else:
        return None, Release.NO_CHANGE, current_version


@attr.s
class Release:
    NO_CHANGE = 'nochanges'
    BREAKING_CHANGE = 'breaking'
    FEATURE = 'feature'
    FIX = 'fix'

    name = attr.ib()
    release_date = attr.ib()
    version = attr.ib()
    description = attr.ib()
    changes = attr.ib(default=attr.Factory(dict))

    @property
    def title(self):
        return


@attr.s
class PullRequest:
    number = attr.ib()
    title = attr.ib()
    description = attr.ib()
    # default is 'body' key
    author = attr.ib()
    labels = attr.ib(default=attr.Factory(list))

    @classmethod
    def from_github(cls, api_response):
        return PullRequest(
            number = api_response['number'],

            title = api_response['title'],

            description = api_response['body'],

            author = api_response['user']['login'],

            labels = [

                label['name']
                for label in api_response['labels']
                # label['colour'] => https://gist.github.com/MicahElliott/719710

            ],
            # labels need a description => map for default github tags
        )


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

    auth_token = attr.ib(default=None)

    @property
    def remote_url(self):
        return git(shlex.split('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(shlex.split(
                'log --oneline --no-color'
            )).split('\n')
            if commit_message
        ]

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

    @property
    def tags(self):
        return git(shlex.split('tag --list')).split('\n')

    @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(shlex.split(
            'log --oneline --merges --no-color{}'.format(revision_range)
        )).split('\n')
        return merge_commits

    # TODO: pull_requests_since(version=None)
    # TODO: cached_property
    @property
    def changes_since_last_version(self):
        pull_requests = []

        for index, commit_msg in enumerate(self.merges_since(self.latest_version)):

            matches = MERGED_PULL_REQUEST.findall(commit_msg)

            if matches:
                _, pull_request_number = matches[0]

                pull_requests.append(PullRequest.from_github(
                    self.github_pull_request(pull_request_number)
                ))
        return pull_requests

    def github_pull_request(self, pr_num):
        pull_request_api_url = uritemplate.expand(
            GITHUB_PULL_REQUEST_API,
            dict(
                owner=self.owner,
                repo=self.repo,
                number=pr_num
            ),
        )

        return requests.get(
            pull_request_api_url,
            headers={
                'Authorization': 'token {}'.format(self.auth_token)
            },
        ).json()

    # TODO: cached_property
    # TODO: move to test fixture
    @property
    def github_labels(self):
        # GET /repos/:owner/:repo/labels
        return {
            'bug': {
                "id": 208045946,
                "url": "https://api.github.com/repos/octocat/Hello-World/labels/bug",

                "name": "bug",
                "color": "f29513",
                "default": True
            },
            'enhancement': {
               "id": 52048165,
               "url": "https://api.github.com/repos/michaeljoseph/changes/labels/enhancement",

               "name": "enhancement",
               "color": "84b6eb",
               "default": True
            },
        }





Push — pyup-initial-update ( 5e7443 )

GitRepository.bitbucket() A

Complexity

Size

Duplication

Code Coverage

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	3	import re
2	3	import shlex
3
4	3	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...
5	3	import semantic_version
		0 ignored issues – show Configuration introduced 2017-10-15 11:30 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...
6	3	import uritemplate
		0 ignored issues – show Configuration introduced 2017-10-15 09:45 UTC by Report Bug Copy Issue Report The import `uritemplate` 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	3	import requests
8	3	import giturlparse
		0 ignored issues – show Configuration introduced 2017-10-14 17:35 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...
9	3	from plumbum.cmd import git
		0 ignored issues – show Configuration introduced 2017-10-15 09:45 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...
10
11	3	MERGED_PULL_REQUEST = re.compile(
12		r'^([0-9a-f]{5,40}) Merge pull request #(\w+)'
13		)
14
15	3	GITHUB_PULL_REQUEST_API = (
16		'https://api.github.com/repos{/owner}{/repo}/issues{/number}'
17		)
18
19
20	3	def changes_to_release_type(repository):
21	3	pull_request_labels = set()
22	3	changes = repository.changes_since_last_version
23
24	3	for change in changes:
25	3	for label in change.labels:
26	3	pull_request_labels.add(label)
27
28	3	change_descriptions = [
29		'\n'.join([change.title, change.description]) for change in changes
30		]
31
32	3	current_version = repository.latest_version
33	3	if 'BREAKING CHANGE' in change_descriptions:
34		return 'major', Release.BREAKING_CHANGE, current_version.next_major()
35	3	elif 'enhancement' in pull_request_labels:
36		return 'minor', Release.FEATURE, current_version.next_minor()
37	3	elif 'bug' in pull_request_labels:
38	3	return 'patch', Release.FIX, current_version.next_patch()
39		else:
40	3	return None, Release.NO_CHANGE, current_version
41
42
43	3	@attr.s
44		class Release:
45	3	NO_CHANGE = 'nochanges'
46	3	BREAKING_CHANGE = 'breaking'
47	3	FEATURE = 'feature'
48	3	FIX = 'fix'
49
50	3	name = attr.ib()
51	3	release_date = attr.ib()
52	3	version = attr.ib()
53	3	description = attr.ib()
54	3	changes = attr.ib(default=attr.Factory(dict))
55
56	3	@property
57		def title(self):
58		return
59
60
61	3	@attr.s
62		class PullRequest:
63	3	number = attr.ib()
64	3	title = attr.ib()
65	3	description = attr.ib()
66		# default is 'body' key
67	3	author = attr.ib()
68	3	labels = attr.ib(default=attr.Factory(list))
69
70	3	@classmethod
71		def from_github(cls, api_response):
72	3	return PullRequest(
73		number = api_response['number'],
		0 ignored issues – show Coding Style introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report No space allowed around keyword argument assignment number = api_response['number'], ^ Loading history...
74		title = api_response['title'],
		0 ignored issues – show Coding Style introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report No space allowed around keyword argument assignment title = api_response['title'], ^ Loading history...
75		description = api_response['body'],
		0 ignored issues – show Coding Style introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report No space allowed around keyword argument assignment description = api_response['body'], ^ Loading history...
76		author = api_response['user']['login'],
		0 ignored issues – show Coding Style introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report No space allowed around keyword argument assignment author = api_response['user']['login'], ^ Loading history...
77		labels = [
		0 ignored issues – show Coding Style introduced 2017-10-18 06:53 UTC by Report Bug Copy Issue Report No space allowed around keyword argument assignment labels = [ ^ Loading history...
78		label['name']
79		for label in api_response['labels']
80		# label['colour'] => https://gist.github.com/MicahElliott/719710
		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 (80/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
81		],
82		# labels need a description => map for default github tags
83		)
84
85
86	3	@attr.s
87		class GitRepository:
88	3	VERSION_ZERO = semantic_version.Version('0.0.0')
89		# TODO: handle multiple remotes (cookiecutter [non-owner maintainer])
90	3	REMOTE_NAME = 'origin'
91
92	3	auth_token = attr.ib(default=None)
93
94	3	@property
95		def remote_url(self):
96	3	return git(shlex.split('config --get remote.{}.url'.format(
97		self.REMOTE_NAME
98		)))
99
100	3	@property
101		def parsed_repo(self):
102	3	return giturlparse.parse(self.remote_url)
103
104	3	@property
105		def repo(self):
106	3	return self.parsed_repo.repo
107
108	3	@property
109		def owner(self):
110	3	return self.parsed_repo.owner
111
112	3	@property
113		def platform(self):
114	3	return self.parsed_repo.platform
115
116	3	@property
117		def is_github(self):
118	3	return self.parsed_repo.github
119
120	3	@property
121		def is_bitbucket(self):
122		return self.parsed_repo.bitbucket
123
124	3	@property
125		def commit_history(self):
126		return [
127		commit_message
128		for commit_message in git(shlex.split(
129		'log --oneline --no-color'
130		)).split('\n')
131		if commit_message
132		]
133
134	3	@property
135		def first_commit_sha(self):
136		return git(
137		'rev-list', '--max-parents=0', 'HEAD'
138		)
139
140	3	@property
141		def tags(self):
142	3	return git(shlex.split('tag --list')).split('\n')
143
144	3	@property
145		def versions(self):
146	3	versions = []
147	3	for tag in self.tags:
148	3	try:
149	3	versions.append(semantic_version.Version(tag))
150	3	except ValueError:
151	3	pass
152	3	return versions
153
154	3	@property
155	3	def latest_version(self) -> semantic_version.Version:
156	3	return max(self.versions) if self.versions else self.VERSION_ZERO
157
158	3	def merges_since(self, version=None):
159	3	if version == semantic_version.Version('0.0.0'):
160		version = self.first_commit_sha
161
162	3	revision_range = ' {}..HEAD'.format(version) if version else ''
163
164	3	merge_commits = git(shlex.split(
165		'log --oneline --merges --no-color{}'.format(revision_range)
166		)).split('\n')
167	3	return merge_commits
168
169		# TODO: pull_requests_since(version=None)
170		# TODO: cached_property
171	3	@property
172		def changes_since_last_version(self):
173	3	pull_requests = []
174
175	3	for index, commit_msg in enumerate(self.merges_since(self.latest_version)):
		0 ignored issues – show Coding Style introduced 2017-10-16 08:16 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...
176	3	matches = MERGED_PULL_REQUEST.findall(commit_msg)
177
178	3	if matches:
179	3	_, pull_request_number = matches[0]
180
181	3	pull_requests.append(PullRequest.from_github(
182		self.github_pull_request(pull_request_number)
183		))
184	3	return pull_requests
185
186	3	def github_pull_request(self, pr_num):
187	3	pull_request_api_url = uritemplate.expand(
188		GITHUB_PULL_REQUEST_API,
189		dict(
190		owner=self.owner,
191		repo=self.repo,
192		number=pr_num
193		),
194		)
195
196	3	return requests.get(
197		pull_request_api_url,
198		headers={
199		'Authorization': 'token {}'.format(self.auth_token)
200		},
201		).json()
202
203		# TODO: cached_property
204		# TODO: move to test fixture
205	3	@property
206		def github_labels(self):
207		# GET /repos/:owner/:repo/labels
208	3	return {
209		'bug': {
210		"id": 208045946,
211		"url": "https://api.github.com/repos/octocat/Hello-World/labels/bug",
		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...
212		"name": "bug",
213		"color": "f29513",
214		"default": True
215		},
216		'enhancement': {
217		"id": 52048165,
218		"url": "https://api.github.com/repos/michaeljoseph/changes/labels/enhancement",
		0 ignored issues – show Coding Style introduced 2017-10-19 21:29 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (94/79). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
219		"name": "enhancement",
220		"color": "84b6eb",
221		"default": True
222		},
223		}
224
225
226
227

michaeljoseph / changes

Push — pyup-initial-update ( 5e7443 )

GitRepository.bitbucket() A

Complexity

Size

Duplication

Code Coverage

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