Skip readthedocs CI if codebase is not affected by PR

[ClaudiaComito]

Due Diligence

• General:
  • title of the PR is suitable to appear in the Release Notes
• Implementation:
  • unit tests: all split configurations tested
  • unit tests: multiple dtypes tested
  • NEW unit tests: MPS tested (1 MPI process, 1 GPU)
  • benchmarks: created for new functionality
  • benchmarks: performance improved or maintained
  • documentation updated where needed

Description

This PR modifies .readthedocs.yaml to skip the ReadTheDocs test build for those PRs that don't modify the codebase (dependencies updates, GH actions etc.)

Background: merging dependabot PRs has become a huge pain because of (among other things) the multiple ReadTheDocs builds trying and failing to run concurrently.

Issue/s resolved: #2070

Changes proposed:

• searches diff for changes in relevant folders (heat/, doc/, examples/, tutorials/). If no changes, cancel readthedocs CI

Type of change

Memory requirements

Performance

Does this change modify the behaviour of other functions? If so, which?

no

[ClaudiaComito]

@JuanPedroGHM I hope this works but can only really test it once merged.

[JuanPedroGHM]

Hi. This seems like a duplicate of #2017? Let's merge this one and close the other one.

[brownbaerchen]

LGTM, thank a lot!

[ClaudiaComito]

Hi. This seems like a duplicate of #2017? Let's merge this one and close the other one.

#2071 , yes you're right, that one didn't work right off the bat and I didn't feel like debugging copilot's code. I just closed #2071

[ClaudiaComito]

Thanks @JuanPedroGHM and @brownbaerchen for the quick approval. The pain is real! 😂 The second approval restarted the CI matrix, which reminds me, that's also a huge pain. Working on it.

[brownbaerchen]

Thanks @JuanPedroGHM and @brownbaerchen for the quick approval. The pain is real! 😂 The second approval restarted the CI matrix, which reminds me, that's also a huge pain. Working on it.

I guess this can also be fixed simply by running the CI on push and PR rather than on approval.

[ClaudiaComito]

Thanks @JuanPedroGHM and @brownbaerchen for the quick approval. The pain is real! 😂 The second approval restarted the CI matrix, which reminds me, that's also a huge pain. Working on it.

I guess this can also be fixed simply by running the CI on push and PR rather than on approval.

Not in its current state, too many jobs, otherwise agreed.