Docs: fix ogp_custom_meta_tags should be a list.

[JulienPalard]

With tuples I get a sphinx warning:

WARNING: The config value `ogp_custom_meta_tags' has type `tuple', defaults to `list'.

followed by a crash:

Traceback (most recent call last):
  File "/home/mdk/src/python/python-docs-fr/.venv/lib/python3.13/site-packages/sphinx/events.py", line 404, in emit
    results.append(listener.handler(self.app, *args))
                   ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
  File "/home/mdk/src/python/python-docs-fr/.venv/lib/python3.13/site-packages/sphinxext/opengraph/__init__.py", line 266, in html_page_context
    context["metatags"] += get_tags(app, context, doctree, app.config)
                           ~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mdk/src/python/python-docs-fr/.venv/lib/python3.13/site-packages/sphinxext/opengraph/__init__.py", line 250, in get_tags
    [make_tag(p, c) for p, c in tags.items()]
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + [make_tag(p, c, "name") for p, c in meta_tags.items()]
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + config["ogp_custom_meta_tags"]
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
TypeError: can only concatenate list (not "tuple") to list

---

📚 Documentation preview 📚: https://cpython-previews--140424.org.readthedocs.build/

[AA-Turner]

Are you using the latest version of sphinxext-opengraph? See sphinx-doc/sphinxext-opengraph#141, this was resolved in 0.13.

A

[bedevere-app]

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

[StanFromIreland]

Per the documentation, it has to be Sequence[str], and only defaults to a tuple.

[StanFromIreland]

The build now fails:

WARNING: The config value ogp_custom_meta_tags' has type list', defaults to `tuple'.

[AA-Turner]

#140425 -- I forgot to update the requirement in CPython, sorry.

A

[JulienPalard]

Are you using the latest version of sphinxext-opengraph? See sphinx-doc/sphinxext-opengraph#141, this was resolved in 0.13.

Checking that.

Per the documentation, it has to be Sequence[str], and only defaults to a tuple.

Ohhh it changed then.

#140425 -- I forgot to update the requirement in CPython, sorry.

OOKKK :)))