canonical-ubuntu-qa team mailing list archive

[Tim Andersson <mp+468925@xxxxxxxxxxxxxxxxxx>]

Tim Andersson has proposed merging ~andersson123/autopkgtest-cloud:generate-charm-inventory-dont-list-docs-entries into autopkgtest-cloud:master.

Commit message:
fix: generate-charm-inventory: don't display docs commits

As pointed out by Brian Murray, changes to the documentation of
autopkgtest-cloud have no effect on the functionality of either charm.

This commit amends the issue by ignoring commits with a message that
begins with "docs:".

Requested reviews:
  Canonical's Ubuntu QA (canonical-ubuntu-qa)

For more details, see:
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/468925

Pruning some less than useful information out of the inventory generated by this script.
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~andersson123/autopkgtest-cloud:generate-charm-inventory-dont-list-docs-entries into autopkgtest-cloud:master.

diff --git a/dev-tools/generate-charm-inventory b/dev-tools/generate-charm-inventory
index 7a9e02a..ff66e55 100755
--- a/dev-tools/generate-charm-inventory
+++ b/dev-tools/generate-charm-inventory
@@ -30,7 +30,9 @@ commits = {
 for commit in repo.iter_commits(branch, max_count=1000):
     if str(commit) == up_to:
         break
-    if commit.message.startswith("Merge "):
+    if commit.message.startswith("Merge ") or commit.message.startswith(
+        "docs:"
+    ):
         continue
     if "web: " in commit.message:
         commits["web"].append(