cookiecutter: Fix adding of an empty keyword

When pressing ENTER while being asked for project keywords, an empty
keyword ("") was added. This commit fixes that.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

RELEASE_NOTES.md

@@ -65,3 +65,5 @@
 ### Cookiecutter template
 
 - Now the CI workflow will checkout the submodules.
+
+- Fix adding of an empty keyword.

cookiecutter/local_extensions.py

@@ -172,7 +172,7 @@ def keywords(cookiecutter: dict[str, str]) -> str:
     cookiecutter_keywords = cookiecutter["keywords"]
     if cookiecutter_keywords == default:
         cookiecutter_keywords = ""
-    extended_keywords.extend(k.strip() for k in cookiecutter_keywords.split(","))
+    extended_keywords.extend(k.strip() for k in cookiecutter_keywords.split(",") if k)
     return _json.dumps(extended_keywords)
 
 

tests_golden/integration/test_cookiecutter_generation/actor/frequenz-actor-test/pyproject.toml

@@ -14,7 +14,7 @@ name = "frequenz-actor-test"
 description = "Test description"
 readme = "README.md"
 license = { text = "MIT" }
-keywords = ["frequenz", "python", "actor", "test", ""]
+keywords = ["frequenz", "python", "actor", "test"]
 classifiers = [ # TODO(cookiecutter): Remove and add more if appropriate
   "Development Status :: 3 - Alpha",
   "Intended Audience :: Developers",

tests_golden/integration/test_cookiecutter_generation/api/frequenz-api-test/pyproject.toml

@@ -14,7 +14,7 @@ name = "frequenz-api-test"
 description = "Test description"
 readme = "README.md"
 license = { text = "MIT" }
-keywords = ["frequenz", "python", "api", "grpc", "protobuf", "rpc", "test", ""]
+keywords = ["frequenz", "python", "api", "grpc", "protobuf", "rpc", "test"]
 classifiers = [ # TODO(cookiecutter): Remove and add more if appropriate
   "Development Status :: 3 - Alpha",
   "Intended Audience :: Developers",

tests_golden/integration/test_cookiecutter_generation/app/frequenz-app-test/pyproject.toml

@@ -14,7 +14,7 @@ name = "frequenz-app-test"
 description = "Test description"
 readme = "README.md"
 license = { text = "MIT" }
-keywords = ["frequenz", "python", "app", "application", "test", ""]
+keywords = ["frequenz", "python", "app", "application", "test"]
 classifiers = [ # TODO(cookiecutter): Remove and add more if appropriate
   "Development Status :: 3 - Alpha",
   "Intended Audience :: Developers",

tests_golden/integration/test_cookiecutter_generation/lib/frequenz-test-python/pyproject.toml

@@ -14,7 +14,7 @@ name = "frequenz-test"
 description = "Test description"
 readme = "README.md"
 license = { text = "MIT" }
-keywords = ["frequenz", "python", "lib", "library", "test", ""]
+keywords = ["frequenz", "python", "lib", "library", "test"]
 classifiers = [ # TODO(cookiecutter): Remove and add more if appropriate
   "Development Status :: 3 - Alpha",
   "Intended Audience :: Developers",

tests_golden/integration/test_cookiecutter_generation/model/frequenz-model-test/pyproject.toml

@@ -14,7 +14,7 @@ name = "frequenz-model-test"
 description = "Test description"
 readme = "README.md"
 license = { text = "MIT" }
-keywords = ["frequenz", "python", "model", "ai", "ml", "machine-learning", "test", ""]
+keywords = ["frequenz", "python", "model", "ai", "ml", "machine-learning", "test"]
 classifiers = [ # TODO(cookiecutter): Remove and add more if appropriate
   "Development Status :: 3 - Alpha",
   "Intended Audience :: Developers",