From 9bfe64d5782ec37939bc922f96c9350fb798f824 Mon Sep 17 00:00:00 2001 From: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com> Date: Fri, 16 Feb 2024 09:59:22 -0500 Subject: [PATCH] Remove qiskit.org links (#419) --- CONTRIBUTING.md | 9 +++------ setup.py | 2 +- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 32d280da..ba493fd0 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -3,7 +3,7 @@ First read the overall project contributing guidelines. These are all included in the qiskit documentation: -https://qiskit.org/documentation/contributing_to_qiskit.html +https://github.com/Qiskit/qiskit/blob/main/CONTRIBUTING.md ## Contributing to QEC @@ -49,9 +49,6 @@ please ensure that: If your pull request is adding a new class, function, or module that is intended to be user facing ensure that you've also added those to a documentation `autosummary` index to include it in the api documentation. - For more details you can refer to: - - https://qiskit.org/documentation/contributing_to_qiskit.html#documentation-structure 3. If it makes sense for your change that you have added new tests that @@ -378,7 +375,7 @@ that minor version on pypi. For example, stable/0.8 contains the code for the merged to it are bugfixes. ## Adding deprecation warnings -The qiskit-qec code is part of Qiskit and, therefore, the [Qiskit Deprecation Policy](https://qiskit.org/documentation/contributing_to_qiskit.html#deprecation-policy) fully applies here. Additionally, qiskit-qec does not allow `DeprecationWarning`s in its testsuite. If you are deprecating code, you should add a test to use the new/non-deprecated method (most of the time based on the existing test of the deprecated method) and alter the existing test to check that the deprecated method still works as expected, [using `assertWarns`](https://docs.python.org/3/library/unittest.html#unittest.TestCase.assertWarns). The `assertWarns` context will silence the deprecation warning while checking that it raises. +The qiskit-qec code is part of the Qiskit ecosystem. So, it chooses to follow the [Qiskit Deprecation Policy](https://github.com/Qiskit/qiskit/blob/main/DEPRECATION.md). Additionally, qiskit-qec does not allow `DeprecationWarning`s in its testsuite. If you are deprecating code, you should add a test to use the new/non-deprecated method (most of the time based on the existing test of the deprecated method) and alter the existing test to check that the deprecated method still works as expected, [using `assertWarns`](https://docs.python.org/3/library/unittest.html#unittest.TestCase.assertWarns). The `assertWarns` context will silence the deprecation warning while checking that it raises. For example, if `Obj.method1` is being deprecated in favour of `Obj.method2`, the existing test (or tests) for `method1` might look like this: @@ -402,7 +399,7 @@ def test_method2(self): self.assertEqual(result, ) ``` -`test_method1_deprecated` can be removed after `Obj.method1` is removed (following the [Qiskit Deprecation Policy](https://qiskit.org/documentation/contributing_to_qiskit.html#deprecation-policy)). +`test_method1_deprecated` can be removed after `Obj.method1` is removed (following the [Qiskit Deprecation Policy](https://github.com/Qiskit/qiskit/blob/main/DEPRECATION.md)). ## Using dependencies diff --git a/setup.py b/setup.py index 044e7b0f..9014389a 100644 --- a/setup.py +++ b/setup.py @@ -156,7 +156,7 @@ def build_extension(self, ext: CMakeExtension) -> None: long_description_content_type="text/markdown", url="https://github.com/Qiskit/qiskit-qec", author="Qiskit Development Team", - author_email="hello@qiskit.org", + author_email="qiskit@us.ibm.com", license="Apache 2.0", classifiers=[ "Environment :: Console",