see what happens if we remove the flaky mark (#1152)

* see what happens if we remove the flaky mark

* now that we run this one slow test on PRs, we should make sure it gets cancled each time a push is made

.github/workflows/package-tests.yaml

@@ -17,6 +17,10 @@ on:
     # At 07:00 UTC on Monday and Thursday.
     - cron: "0 7 * * *"
 
+concurrency:
+  group: "${{ github.workflow }}-${{ github.ref }}"
+  cancel-in-progress: true
+
 defaults:
   run:
     shell: bash -leo pipefail {0}

openfe/tests/protocols/test_openmmutils.py

@@ -931,7 +931,7 @@ def test_openeye_import_error(self, monkeypatch, uncharged_mol):
 @pytest.mark.slow
 @pytest.mark.download
 # Sometimes we get a DOI lookup error from duecredit
-@pytest.mark.flaky(reruns=3, only_rerun=ValueError, reruns_delay=10)
+#@pytest.mark.flaky(reruns=3, only_rerun=ValueError, reruns_delay=10)
 def test_forward_backwards_failure(simulation_nc):
     rep = multistate.multistatereporter.MultiStateReporter(
         simulation_nc,