canonical-ubuntu-qa team mailing list archive

[Skia <mp+461964@xxxxxxxxxxxxxxxxxx>]

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.