Repository: PennyLaneAI/pennylane-rigetti (language: python)

Issue:
Fix missing comma
In #119, we noted that the missing comma in `qcs-api-client>=0.20.13<0.22.0` in `setup.py` was causing issues. With the changes here, the reported problem disappears:

```python
>>> import pennylane as qml
>>> qml.device("rigetti.qpu", device="Aspen-M-2")
<QPUDevice device (wires=80, shots=1000) at 0x132e949d0>
```

What's more, with `pip install -e '.[test]'` in a virtual environment, all unit tests pass (with some warnings).


Discussion / hints:
@AlbertMitjans mind taking a look at this?
Sorry, I hadn't looked at `.github/workflows` before making my PR! Hopefully 2583c76 helps.
`pennylane` no longer supports `python3.7`. [I opened a PR](https://github.com/PennyLaneAI/pennylane-rigetti/pull/121) where I update the python version used in the docs and in the integration tests.
@MarquessV @genos do you have permission to review my PR? Otherwise I'll notify someone from the PL team.
> @MarquessV @genos do you have permission to review my PR? Otherwise I'll notify someone from the PL team.

Sorry! I’m happy to take a look, but I lack GH permissions to give any official review.
@AlbertMitjans in the last 3 commits, I reinstated `requirements.txt`. What's more, I noticed that the GitHub workflows were using different & out-of-date `quilc` versions, plus the test requirements were specified in multiple places; dca8a92 hopes to fix that.
Perhaps also updating the `quilc` versions in testing was a bit beyond the scope of this PR 😂😬 Sorry! If tests fail again, I’ll set that versioning back and restrict the scope here to just `pip` usage.

Fix patch:
diff --git a/.github/workflows/post_release_version_bump.yml b/.github/workflows/post_release_version_bump.yml
index 56004f71..0c866839 100644
--- a/.github/workflows/post_release_version_bump.yml
+++ b/.github/workflows/post_release_version_bump.yml
@@ -35,5 +35,5 @@ jobs:
           title: Version Bump
           body: updated changelog and _version.py
           branch: post-release-version-bump
-          reviewers: antalszava
+          reviewers: timmysilv
           base: master
diff --git a/.github/workflows/pre_release_version_bump.yml b/.github/workflows/pre_release_version_bump.yml
index 1ac68282..412fd3b4 100644
--- a/.github/workflows/pre_release_version_bump.yml
+++ b/.github/workflows/pre_release_version_bump.yml
@@ -34,5 +34,5 @@ jobs:
           title: Version Bump
           body: updated changelog and _version.py
           branch: pre-release-version-bump
-          reviewers: antalszava
+          reviewers: timmysilv
           base: master
diff --git a/CHANGELOG.md b/CHANGELOG.md
index df8f2935..76dc3aaa 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,11 +4,8 @@
 
 ### Breaking changes
 
-* Build dependency specification is now solely in `requirements.txt`, which is read
-  by `setup.py`. What's more, a missing comma has been added to the version bounds
-  on `qcs-api-client`, fixing ([#120](https://github.com/PennyLaneAI/pennylane-rigetti/pull/120))
-  a bug ([#119](https://github.com/PennyLaneAI/pennylane-rigetti/issues/119)) that
-  was causing errors when trying to use the package.
+* Build dependency specification is now solely in `requirements.txt`, which is read by `setup.py`.
+  [(#120)](https://github.com/PennyLaneAI/pennylane-rigetti/pull/120)
 
 * `pennylane-rigetti` no longer supports `python3.7`. A newer python version is required.
   [(#121)](https://github.com/PennyLaneAI/pennylane-rigetti/pull/121)
@@ -22,12 +19,16 @@
 
 ### Bug fixes
 
+* A missing comma has been added to the version bounds on `qcs-api-client`.
+  [(#120)](https://github.com/PennyLaneAI/pennylane-rigetti/pull/120)
+
 ### Contributors
 
 This release contains contributions from (in alphabetical order):
 
 Albert Mitjans Coma
 Graham Enos
+Mudit Pandey
 
 ---
 


Test patch:
diff --git a/tests/test_qvm.py b/tests/test_qvm.py
index b36b1905..4fbd4b1f 100644
--- a/tests/test_qvm.py
+++ b/tests/test_qvm.py
@@ -612,7 +612,7 @@ def test_apply_qubitstatesvector_raises_an_error_if_not_first(self):
         applied as the first operation."""
         dev = qml.device("rigetti.qvm", device="2q-qvm", parametric_compilation=True)
 
-        operation = qml.QubitStateVector(np.array([1, 0]), wires=list(range(2)))
+        operation = qml.QubitStateVector(np.array([1, 0]), wires=[0])
         queue = [qml.PauliX(0), operation]
         with pytest.raises(
             qml.DeviceError,