I am trying to work on finding the following metrics
New Work - totally new code which does not replace other code.
Churn - code that is rewritten or deleted after being written
Help Others - where developer helps on someone else work
While I was working on churn and new work metrics, I cam to know about legacy refactor which says it's a code that updates or edits old code.
I am confused in between churn and legacy refactor. To me it seems like the same thing.
For calculating churn the code(taking help from truegitcodechurn) is below
def get_commits_by_author(self, author, after=None, before=None, format='%h'):
cmd = f'git log --author="{author}" --format="{format}" --no-abbrev '
cmd += f'--before="{before}" --after="{after}" '
cmd += f'{self.no_merges} {get_log_range("develop")} --reverse'
command = [f'{cmd}']
result = get_proc_out(command).split('\n')
self.commits_by_author = result
return self.commits_by_author
def get_churn_per_author(self, author, before, after, exdirs=[]):
commits = get_commits_by_author(self, author, before, after, format="%h")
files = {}
contribution = 0
churn = 0
contributors = author
for commit in commits:
[files, contribution, churn] = get_loc(
commit, files, contribution, churn, exdirs
)
if len(contributors.strip()) == 0:
contributors = set(
get_commits_by_author(self, author, before, after, "%an")
).__str__()
contributors = contributors.replace("{", "").replace("}", "").replace("'", "")
return contributors, contribution, churn
def get_loc(commit, files, contribution, churn, exdirs):
cmd = f"git show --format= --unified=0 --no-prefix {commit}"
if len(exdirs) > 1:
cmd += " -- ."
for exdir in exdirs:
cmd += f' ":!{exdir}"'
print("cmd", cmd)
command = [f"{cmd}"]
results = get_proc_out(command).splitlines()
file = ""
loc_changes = ""
for result in results:
new_file = is_new_file(result, file)
if file != new_file:
file = new_file
if file not in files:
files[file] = {}
else:
# @@ -1,5 +1,4 @@ = -1,5 +1,4
new_loc_changes = is_loc_change(result, loc_changes)
if loc_changes != new_loc_changes:
loc_changes = new_loc_changes
locc = get_loc_change(loc_changes)
# locc = {2: 0} or {8: 0, 9: 1}
for loc in locc:
# files[file] = {2: 0, 8: 0, 9: 1}
if loc in files[file]:
files[file][loc] += locc[loc]
churn += abs(locc[loc])
else:
files[file][loc] = locc[loc]
contribution += abs(locc[loc])
else:
continue
return [files, contribution, churn]
def get_loc_change(loc_changes):
# removals
# -1,5 +1,4 = -1,5
left = loc_changes[: loc_changes.find(" ")]
left_dec = 0
# 2
if left.find(",") > 0:
# 2
comma = left.find(",")
# 5
left_dec = int(left[comma + 1 :])
# 1
left = int(left[1:comma])
else:
left = int(left[1:])
left_dec = 1
# additions
# +1,4
right = loc_changes[loc_changes.find(" ") + 1 :]
right_dec = 0
if right.find(",") > 0:
comma = right.find(",")
right_dec = int(right[comma + 1 :])
right = int(right[1:comma])
else:
right = int(right[1:])
right_dec = 1
if left == right:
return {left: (right_dec - left_dec)}
else:
return {left: left_dec, right: right_dec}
def is_loc_change(result, loc_changes):
# search for loc changes (@@ ) and update loc_changes variable
# @@ -1,5 +1,4 @@
if result.startswith("@@"):
# loc_change = result[2+1: ] -> -1,5 +1,4 @@
loc_change = result[result.find(" ") + 1 :]
# loc_change = loc_change[:9] -> -1,5 +1,4
loc_change = loc_change[: loc_change.find(" @@")]
return loc_change
else:
return loc_changes
def is_new_file(result, file):
# search for destination file (+++ ) and update file variable
if result.startswith("+++"):
return result[result.rfind(" ") + 1 :]
else:
return file
Firstly, I want to understand what is the difference between them, and if legacy refactor is different than churn, then what should I consider to separate churn and legacy refactor result from the above code so that it can be shown clearly that this is churn and this is legacy refactor.
