diff --git a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/errorprone/UnusedAssignmentRule.java b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/UnusedAssignmentRule.java
similarity index 99%
rename from pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/errorprone/UnusedAssignmentRule.java
rename to pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/UnusedAssignmentRule.java
index 3822ce73f7..536d7d2816 100644
--- a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/errorprone/UnusedAssignmentRule.java
+++ b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/UnusedAssignmentRule.java
@@ -2,7 +2,7 @@
  * BSD-style license; for more info see http://pmd.sourceforge.net/license.html
  */
 
-package net.sourceforge.pmd.lang.java.rule.errorprone;
+package net.sourceforge.pmd.lang.java.rule.bestpractices;
 
 
 import static net.sourceforge.pmd.lang.java.rule.codestyle.ConfusingTernaryRule.unwrapParentheses;
diff --git a/pmd-java/src/main/resources/category/java/bestpractices.xml b/pmd-java/src/main/resources/category/java/bestpractices.xml
index 9c2dbf7138..a295c1b48b 100644
--- a/pmd-java/src/main/resources/category/java/bestpractices.xml
+++ b/pmd-java/src/main/resources/category/java/bestpractices.xml
@@ -1298,6 +1298,120 @@ class Foo{
         </example>
     </rule>
 
+    <rule name="UnusedAssignment"
+          language="java"
+          since="6.26.0"
+          message="The value assigned to this variable is never used or always overwritten"
+          class="net.sourceforge.pmd.lang.java.rule.bestpractices.UnusedAssignmentRule"
+          externalInfoUrl="${pmd.website.baseurl}/pmd_rules_java_bestpractices.html#unusedassignment">
+        <description>
+            Reports assignments to variables that are never used before the variable is overwritten,
+            or goes out of scope. Unused assignments are those for which
+            1. The variable is never read after the assignment, or
+            2. The assigned value is always overwritten by other assignments before the next read of
+            the variable.
+
+            The rule doesn't consider assignments to fields except for those of `this` in a constructor,
+            or static fields of the current class in static initializers.
+
+            The rule may be suppressed with the standard `@SuppressWarnings("unused")` tag.
+
+            The rule subsumes UnusedLocalVariable, and UnusedFormalParameter. Those violations are filtered
+            out by default, in case you already have enabled those rules, but may be enabled with the property
+            `reportUnusedVariable`. Variables whose name starts with `ignored` are filtered out, as
+            is standard practice for exceptions.
+        </description>
+        <priority>3</priority>
+        <example>
+            <![CDATA[
+            class A {
+                // this field initializer is redundant,
+                // it is always overwritten in the constructor
+                int f = 1;
+
+                A(int f) {
+                    this.f = f;
+                }
+            }
+        ]]>
+        </example>
+        <example><![CDATA[
+class B {
+
+    int method(int i, int j) {
+        // this initializer is redundant,
+        // it is overwritten in all branches of the `if`
+        int k = 0;
+
+        // Both the assignments to k are unused, because k is
+        // not read after the if/else
+        // This may hide a bug: the programmer probably wanted to return k
+        if (i < j)
+            k = i;
+        else
+            k = j;
+
+        return j;
+    }
+
+}
+        ]]>
+
+        </example>
+        <example><![CDATA[
+class C {
+
+    int method() {
+        int i = 0;
+
+        checkSomething(++i);
+        checkSomething(++i);
+        checkSomething(++i);
+        checkSomething(++i);
+
+        // That last increment is not reported unless
+        // the property `checkUnusedPrefixIncrement` is
+        // set to `true`
+        // Technically it could be written (i+1), but it
+        // is not very important
+    }
+
+}
+        ]]>
+
+        </example>
+        <example><![CDATA[
+class C {
+
+    // variables that are truly unused (at most assigned to, but never accessed)
+    // are only reported if property `reportUnusedVariables` is true
+
+    void method(int param) { } // for example this method parameter
+
+    // even then, you can suppress the violation with an annotation:
+
+    void method(@SuppressWarning("unused") int param) { } // no violation, even if `reportUnusedVariables` is true
+
+    // For catch parameters, or for resources which don't need to be used explicitly,
+    // you can give a name that starts with "ignored" to ignore such warnings
+
+    {
+        try (Something ignored = Something.create()) {
+            // even if ignored is unused, it won't be flagged
+            // its purpose might be to side-effect in the create/close routines
+
+        } catch (Exception e) { // this is unused and will cause a warning if `reportUnusedVariables` is true
+            // you should choose a name that starts with "ignored"
+            return;
+        }
+    }
+
+}
+        ]]>
+
+        </example>
+    </rule>
+
     <rule name="UnusedFormalParameter"
           language="java"
           since="0.8"
diff --git a/pmd-java/src/main/resources/category/java/errorprone.xml b/pmd-java/src/main/resources/category/java/errorprone.xml
index 94a7f83d25..8033820e7f 100644
--- a/pmd-java/src/main/resources/category/java/errorprone.xml
+++ b/pmd-java/src/main/resources/category/java/errorprone.xml
@@ -3276,122 +3276,6 @@ public String convert(int x) {
         </example>
     </rule>
 
-
-    <rule name="UnusedAssignment"
-          language="java"
-          since="6.26.0"
-          message="The value assigned to this variable is never used or always overwritten"
-          class="net.sourceforge.pmd.lang.java.rule.errorprone.UnusedAssignmentRule"
-          externalInfoUrl="${pmd.website.baseurl}/pmd_rules_java_errorprone.html#unusedassignment">
-        <description>
-            Reports assignments to variables that are never used before the variable is overwritten,
-            or goes out of scope. Unused assignments are those for which
-            1. The variable is never read after the assignment, or
-            2. The assigned value is always overwritten by other assignments before the next read of
-            the variable.
-
-            The rule doesn't consider assignments to fields except for those of `this` in a constructor,
-            or static fields of the current class in static initializers.
-
-            The rule may be suppressed with the standard `@SuppressWarnings("unused")` tag.
-
-            The rule subsumes UnusedLocalVariable, and UnusedFormalParameter. Those violations are filtered
-            out by default, in case you already have enabled those rules, but may be enabled with the property
-            `reportUnusedVariable`. Variables whose name starts with `ignored` are filtered out, as
-            is standard practice for exceptions.
-        </description>
-        <priority>3</priority>
-        <example>
-        <![CDATA[
-            class A {
-                // this field initializer is redundant,
-                // it is always overwritten in the constructor
-                int f = 1;
-
-                A(int f) {
-                    this.f = f;
-                }
-            }
-        ]]>
-        </example>
-        <example><![CDATA[
-class B {
-
-    int method(int i, int j) {
-        // this initializer is redundant,
-        // it is overwritten in all branches of the `if`
-        int k = 0;
-
-        // Both the assignments to k are unused, because k is
-        // not read after the if/else
-        // This may hide a bug: the programmer probably wanted to return k
-        if (i < j)
-            k = i;
-        else
-            k = j;
-
-        return j;
-    }
-
-}
-        ]]>
-
-        </example>
-        <example><![CDATA[
-class C {
-
-    int method() {
-        int i = 0;
-
-        checkSomething(++i);
-        checkSomething(++i);
-        checkSomething(++i);
-        checkSomething(++i);
-
-        // That last increment is not reported unless
-        // the property `checkUnusedPrefixIncrement` is
-        // set to `true`
-        // Technically it could be written (i+1), but it
-        // is not very important
-    }
-
-}
-        ]]>
-
-        </example>
-        <example><![CDATA[
-class C {
-
-    // variables that are truly unused (at most assigned to, but never accessed)
-    // are only reported if property `reportUnusedVariables` is true
-
-    void method(int param) { } // for example this method parameter
-
-    // even then, you can suppress the violation with an annotation:
-
-    void method(@SuppressWarning("unused") int param) { } // no violation, even if `reportUnusedVariables` is true
-
-    // For catch parameters, or for resources which don't need to be used explicitly,
-    // you can give a name that starts with "ignored" to ignore such warnings
-
-    {
-        try (Something ignored = Something.create()) {
-            // even if ignored is unused, it won't be flagged
-            // its purpose might be to side-effect in the create/close routines
-
-        } catch (Exception e) { // this is unused and will cause a warning if `reportUnusedVariables` is true
-            // you should choose a name that starts with "ignored"
-            return;
-        }
-    }
-
-}
-        ]]>
-
-        </example>
-    </rule>
-
-
     <rule name="UnusedNullCheckInEquals"
           language="java"
           since="3.5"
diff --git a/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/errorprone/UnusedAssignmentTest.java b/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/bestpractices/UnusedAssignmentTest.java
similarity index 78%
rename from pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/errorprone/UnusedAssignmentTest.java
rename to pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/bestpractices/UnusedAssignmentTest.java
index 89ddb757ae..7d363ef084 100644
--- a/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/errorprone/UnusedAssignmentTest.java
+++ b/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/bestpractices/UnusedAssignmentTest.java
@@ -1,8 +1,8 @@
-/**
+/*
  * BSD-style license; for more info see http://pmd.sourceforge.net/license.html
  */
 
-package net.sourceforge.pmd.lang.java.rule.errorprone;
+package net.sourceforge.pmd.lang.java.rule.bestpractices;
 
 import net.sourceforge.pmd.testframework.PmdRuleTst;
 
diff --git a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/errorprone/xml/UnusedAssignment.xml b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/bestpractices/xml/UnusedAssignment.xml
similarity index 100%
rename from pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/errorprone/xml/UnusedAssignment.xml
rename to pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/bestpractices/xml/UnusedAssignment.xml