Run full API docs invoke task for every PR

[ml-evs]

Closes #747.

It looks like the Elasticsearch docs will need to be fixed as part of this PR as it uses a deferred type hint. This may require switching to Python 3.9 for docs builds.

[codecov]

Codecov Report

Merging #748 (46c9a8f) into master (4f503eb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #748   +/-   ##
=======================================
  Coverage   92.79%   92.79%           
=======================================
  Files          68       68           
  Lines        3663     3663           
=======================================
  Hits         3399     3399           
  Misses        264      264           

Flag	Coverage Δ
project	92.79% <ø> (ø)
validator	92.79% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
optimade/server/entry_collections/elasticsearch.py	97.91% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f503eb...46c9a8f. Read the comment docs.

[ml-evs]

Hmmm, so we can't use deferred type hints with Python 3.6. Is it time to drop Python 3.6 support, or is it not worth it for the sake of an optional type hint?

[CasperWA]

Hmmm, so we can't use deferred type hints with Python 3.6. Is it time to drop Python 3.6 support, or is it not worth it for the sake of an optional type hint?

I think we should definitely start considering dropping Python 3.6 support.

[CasperWA]

Fixed it - no reason to look to the future 😅

[ml-evs]

Fixed it - no reason to look to the future sweat_smile

Ah, did you test this locally with mkdocs too? I was under the impression it wasn't supported

[CasperWA]

Ah, did you test this locally with mkdocs too? I was under the impression it wasn't supported

Yup. And it looks all right in the produced documentation as well :)

[ml-evs]

Nice, let's punch it then

[CasperWA]

It's a rebase+merge (or similar) from here on, I think.