From 82a57ef980adba4a101e201d6d3a3a3583cc9f2b Mon Sep 17 00:00:00 2001
From: Ihar Hrachyshka <ihar.hrachyshka@gmail.com>
Date: Mon, 11 Nov 2024 13:31:39 -0500
Subject: [PATCH] refactor: Move hacking dependency from tox.ini to test-reqs

This dependency was moved from test-requirements.txt to tox.ini[deps] a
long time ago: I0111c41bea6a6caf5ffba1f5c34489854d9c9747 due to some pip
resolver issues related to lower-constraints.txt

Since then, a lot has changed and this probably doesn't apply anymore
(other projects like nova already keep the dependency in
test-requirements.txt).

We actually import hacking from neutron.tests.unit (because we have
custom rules), so it makes more sense to keep the dependency with the
rest of requirements.

Change-Id: I02ad072fb2982f6968db987cb822efdc1b6f10ba
---
 test-requirements.txt | 1 +
 tox.ini               | 1 -
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/test-requirements.txt b/test-requirements.txt
index eeedc0de47f..a35a4a18250 100644
--- a/test-requirements.txt
+++ b/test-requirements.txt
@@ -1,3 +1,4 @@
+hacking>=6.1.0,<6.2.0 # Apache-2.0
 coverage>=4.4.1 # Apache-2.0
 fixtures>=3.0.0 # Apache-2.0/BSD
 python-subunit>=1.0.0 # Apache-2.0/BSD
diff --git a/tox.ini b/tox.ini
index 7250ddb4d4a..d6afca47a94 100644
--- a/tox.ini
+++ b/tox.ini
@@ -28,7 +28,6 @@ deps =
   -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}
   -r{toxinidir}/requirements.txt
   -r{toxinidir}/test-requirements.txt
-  hacking>=6.1.0,<6.2.0 # Apache-2.0
 allowlist_externals = bash
 commands =
   bash {toxinidir}/tools/pip_install_src_modules.sh "{toxinidir}"