Diff comments:
> diff --git a/dev-tools/generate-charm-inventory
> b/dev-tools/generate-charm-inventory
> index 2aa5785..2f2eb28 100755
> --- a/dev-tools/generate-charm-inventory
> +++ b/dev-tools/generate-charm-inventory
> @@ -26,12 +26,14 @@ commits = {
> "cloud": [],
> "misc": [],
> }
> +changed_files = set(())
`set()` should be enough, or did I miss something?
>
> for commit in repo.iter_commits(branch, max_count=1000):
> if str(commit) == up_to:
> break
> - if "Merge branch" in commit.message:
> + if str(commit.message).startswith("Merge "):
> continue
> + changed_files.update(commit.stats.files)
> if "web: " in commit.message:
> commits["web"].append(
> "- [%s](%s%s)"
--
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/461964
Your team Canonical's Ubuntu QA is requested to review the proposed merge of
~andersson123/autopkgtest-cloud:charm-inventory-amendments into
autopkgtest-cloud:master.
--
Mailing list: https://launchpad.net/~canonical-ubuntu-qa
Post to : canonical-ubuntu-qa@lists.launchpad.net
Unsubscribe : https://launchpad.net/~canonical-ubuntu-qa
More help : https://help.launchpad.net/ListHelp
