Skip integraton tests on draft PRs

[jdangerx]

To save time/energy/$, we can only run the long CI jobs on PRs that are nominally ready for review.

Testing plan:

• make this a draft PR, and see if ci-integration runs
• turn this into a non-draft PR, and see if ci-integration runs

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[jdangerx]

Boo, it doesn't re-run once you switch to ready-for-review. Need to look into that a bit more.

[zaneselvans]

Might we want to add a workflow_dispatch here so someone can manually kick off the integration tests once without needing to change the status of the PR?

[jdangerx]

Maybe. I'm not sure how the workflow_dispatch event will interact with the if: expression, since then presumably pull_request.event won't be defined.

I'm inclined to keep this change as-is and not deal with that, since it's pretty easy to change the status of the PR to get integration tests running if one wants them.

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (da51425) 88.5% compared to head (b3567e5) 88.5%.

Additional details and impacted files

@@          Coverage Diff          @@
##             dev   #2839   +/-   ##
=====================================
  Coverage   88.5%   88.5%           
=====================================
  Files         90      90           
  Lines      10134   10134           
=====================================
  Hits        8978    8978           
  Misses      1156    1156           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bendnorman]

This is great thanks @jdangerx !