Warning about natural language literals

[OriolAbril]

It might be related to #36 but I think it is an unintended consequence of #37. I am now seeing a warning about natural language literals, even in the example package which I think makes sensible use of literals. It probably happens in tests too:

WARNING: natural language literal with one item 1, consider using Literal[1, 3, "foo"] to improve readability
WARNING: natural language literal with one item "uno", consider using Literal["uno", 2, "drei", "four"] to improve readability

[lagru]

Uff, good catch. My bad. I made a typo in the logic guarding this warning:

docstub/src/docstub/_docstrings.py

Lines 384 to 390 in 7f02a0e

	if len(tree.children):
	logger.warning(
	"natural language literal with one item `%s`, "
	"consider using `%s` to improve readability",
	tree.children[0],
	out,
	)

Should be if len(tree.children) == 1! Let me know if you want the dubious honor of fixing it since you found it? 😄

[lagru]

Long-term we really need a better story to surface errors, warnings and the like in tests. I'd like to use a proper logging library to figure out reporting errors with line context. But so far haven't had the time / priority to figure it out. Instead I'm using something handcrafted for now:

docstub/src/docstub/_utils.py

Line 132 in 7f02a0e

class ErrorReporter:

[lagru]

Only now seeing #40. 🙈