Commit 3ecbe83

committed

refactor(git): retype get_commits parameter to make it more friendly to call sites

1 parent a9cd957 commit 3ecbe83Copy full SHA for 3ecbe83

File tree

-7

lines changed

-7

lines changed

Lines changed: 1 addition & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -253,10 +253,7 @@ def __call__(self) -> None:`
`253`	`253`	`) from exc`
`254`	`254`	`else:`
`255`	`255`	`if increment is None:`
`256`		`- if current_tag:`
`257`		`- commits = git.get_commits(current_tag.name)`
`258`		`- else:`
`259`		`- commits = git.get_commits()`
	`256`	`+ commits = git.get_commits(current_tag.name if current_tag else None)`
`260`	`257`
`261`	`258`	`# No commits, there is no need to create an empty tag.`
`262`	`259`	`# Unless we previously had a prerelease.`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ def _get_commits(self) -> list[git.GitCommit]:`
`110`	`110`	`return [git.GitCommit(rev="", title="", body=self._filter_comments(msg))]`
`111`	`111`
`112`	`112`	`# Get commit messages from git log (--rev-range)`
`113`		`- return git.get_commits(end=self.rev_rangeor"HEAD")`
	`113`	`+ return git.get_commits(end=self.rev_range)`
`114`	`114`
`115`	`115`	`@staticmethod`
`116`	`116`	`def _filter_comments(msg: str) -> str:`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -203,15 +203,18 @@ def _create_commit_cmd_string(args: str, committer_date: str \| None, name: str)`
`203`	`203`
`204`	`204`	`def get_commits(`
`205`	`205`	`start: str \| None = None,`
`206`		`- end: str ="HEAD",`
	`206`	`+ end: str \|None=None,`
`207`	`207`	`*,`
`208`	`208`	`args: str = "",`
`209`	`209`	`) -> list[GitCommit]:`
`210`	`210`	`"""Get the commits between start and end."""`
	`211`	`+ if end is None:`
	`212`	`+ end = "HEAD"`
`211`	`213`	`git_log_entries = _get_log_as_str_list(start, end, args)`
`212`	`214`	`return [`
`213`	`215`	`GitCommit.from_rev_and_commit(rev_and_commit)`
`214`		`- for rev_and_commit in filter(None, git_log_entries)`
	`216`	`+ for rev_and_commit in git_log_entries`
	`217`	`+ if rev_and_commit`
`215`	`218`	`]`
`216`	`219`
`217`	`220`

Comments

(0)