Repository: NREL/rdtools (language: python)

Issue:
The number of CI jobs is a bit excessive
Between linting, pytest, environment consistency, and now notebook checks, RdTools PRs have 25-30 CI jobs nowadays.  However, a big chunk of that is because the pytest jobs are doubled up -- they run on both `pull_request` and `push` ([ref](https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request)).  I think running `push` on PRs is unnecessary: it's running the tests on the latest commit of the PR branch, but really we're more interested in what would happen if the PR branch were merged into the target branch (which is what `pull_request` does).  However, `push` might still be useful outside of the context of a PR, for example if we pushed straight to `master` or something.  

I did a brief survey of what other packages do and found that a good number of them restrict the `push` to only apply to a select few important branches like main/master.  That means that `push` would not apply to a regular feature branch used to open a PR, and so the CI jobs on the PR would not be doubled up.  Here are the packages I found in a few minutes of searching: [astropy](https://github.com/astropy/astropy/blob/main/.github/workflows/ci_workflows.yml), [metpy](https://github.com/Unidata/MetPy/blob/main/.github/workflows/tests-pypi.yml), [geopandas](https://github.com/geopandas/geopandas/blob/main/.github/workflows/tests.yaml), [numpy](https://github.com/numpy/numpy/blob/main/.github/workflows/build_test.yml), [pandas](https://github.com/pandas-dev/pandas/blob/main/.github/workflows/code-checks.yml), [scipy](https://github.com/scipy/scipy/blob/main/.github/workflows/linux.yml).

I suggest we take that approach here, i.e. change this line in the pytest job config:

https://github.com/NREL/rdtools/blob/074909079bcb6c9caa6ca63a2e2719edce432b6c/.github/workflows/pytest.yaml#L3

to:

```yaml
on:
  push:
    branches:
    - master
    - development
  pull_request:
```

I think I'd probably support doing that for the flake8 and requirements jobs too.


Discussion / hints:

Fix patch:
diff --git a/.github/workflows/requirements.yaml b/.github/workflows/requirements.yaml
index 690500dc..4bf39728 100644
--- a/.github/workflows/requirements.yaml
+++ b/.github/workflows/requirements.yaml
@@ -1,6 +1,12 @@
 name: requirements
 
-on: [pull_request, push]
+on:
+  push:
+    branches:
+    - master
+    - development
+  pull_request:
+
 
 jobs:
   requirements:
diff --git a/docs/sphinx/source/changelog/pending.rst b/docs/sphinx/source/changelog/pending.rst
index f3f4f226..95e46e5c 100644
--- a/docs/sphinx/source/changelog/pending.rst
+++ b/docs/sphinx/source/changelog/pending.rst
@@ -11,3 +11,5 @@ Requirements
 * Bump ``ipython==7.16.3``, ``jupyter-console==6.4.0``,
   and ``prompt-toolkit==3.0.27`` in ``docs/notebook_requirements.txt``
   and bump ``Pillow==9.0.0`` in ``requirements.txt`` (:pull:`314`)
+* Bump ``nbsphinx`` version from 0.8.5 to 0.8.8 in the optional
+  ``[doc]`` requirements (:pull:`317`)
diff --git a/setup.py b/setup.py
index 5eb8cbff..add256aa 100644
--- a/setup.py
+++ b/setup.py
@@ -56,7 +56,7 @@
 EXTRAS_REQUIRE = {
     'doc': [
         'sphinx==3.2',
-        'nbsphinx==0.8.5',
+        'nbsphinx==0.8.8',
         'nbsphinx-link==1.3.0',
         'sphinx_rtd_theme==0.5.2',
         'ipython',


Test patch:
diff --git a/.github/workflows/pytest.yaml b/.github/workflows/pytest.yaml
index 70c869e1..393a5d43 100644
--- a/.github/workflows/pytest.yaml
+++ b/.github/workflows/pytest.yaml
@@ -1,6 +1,12 @@
 name: pytest
 
-on: [pull_request, push]
+on:
+  push:
+    branches:
+    - master
+    - development
+  pull_request:
+
 
 jobs:
   build: