You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I can reproduce this on main. The keywords answer is split on , and not trimmed, so kw1, second kw gives keywords = ["kw1"," second kw"], and an empty answer gives keywords = [""]. The CITATION.cff template has the same loop, and an empty answer writes a keywords: list with one empty entry.
What I'd change:
trim each keyword and drop empty ones, in both pyproject.toml.jinja and the CITATION.cff template
leave keywords: out of CITATION.cff when nothing is left, and write keywords = [] in pyproject.toml
two tests in tests/test_values.py that fail on main and pass with the change, plus a CHANGELOG line
Would you welcome a PR for this? I have it ready.
Activity
[-]handles spaces between keywords[/-][+]handle spaces between keywords[/+]simonvanlierde commentedon Oct 26, 2026
Hi,
I can reproduce this on
main. Thekeywordsanswer is split on,and not trimmed, sokw1, second kwgiveskeywords = ["kw1"," second kw"], and an empty answer giveskeywords = [""]. The CITATION.cff template has the same loop, and an empty answer writes akeywords:list with one empty entry.What I'd change:
pyproject.toml.jinjaand the CITATION.cff templatekeywords:out of CITATION.cff when nothing is left, and writekeywords = []inpyproject.tomltests/test_values.pythat fail onmainand pass with the change, plus a CHANGELOG lineWould you welcome a PR for this? I have it ready.
Simon
sjvrijn commentedon Oct 26, 2026
@simonvanlierde yes, a PR would be very welcome!
sjvrijn commentedon Oct 28, 2026
Closed by #715