Build config: validate that elements of extra_requirements are strings

[supersergiy]

Details:

• Project URL: https://readthedocs.org/projects/zetta_utils/
• Build URL(if applicable): https://readthedocs.org/projects/zetta-utils/builds/17570682/
• Read the Docs username(if applicable): supersergiy

Expected Result

Docs to build or give a meaningful error message

Actual Result

Docs did not build or give a meaningful error message

[humitos]

I think I found the issue with this build.

It seems that you are specifying extra_requirements as [docs] when it should be only docs in this line https://github.com/ZettaAI/zetta_utils/blob/99a103068a6a8821ddff396054562ec3099eef68/.readthedocs.yml#L27. Can you please change that line and let me know if that works?

We should validate this in our config file and show a better error message to the user. We are validating python.extra_requirements to be a list, but we should also validate that each item of that list is a string in this part of the code:

readthedocs.org/readthedocs/config/config.py

Lines 520 to 528 in 7e0fa4e

	# Validate extra_requirements.
	if 'extra_requirements' in raw_python:
	raw_extra_requirements = raw_python['extra_requirements']
	if not isinstance(raw_extra_requirements, list):
	self.error(
	'python.extra_requirements',
	self.PYTHON_EXTRA_REQUIREMENTS_INVALID_MESSAGE,
	code=PYTHON_INVALID,
	)

cc @stsewd