diff --git a/base/uk.ac.stfc.isis.ibex.client.tycho.parent/checkstyle.xml b/base/uk.ac.stfc.isis.ibex.client.tycho.parent/checkstyle.xml
index 49a504624b..75b8ee2ce5 100644
--- a/base/uk.ac.stfc.isis.ibex.client.tycho.parent/checkstyle.xml
+++ b/base/uk.ac.stfc.isis.ibex.client.tycho.parent/checkstyle.xml
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<!DOCTYPE module PUBLIC "-//Checkstyle//DTD Check Configuration 1.3//EN" "https://checkstyle.org/dtds/configuration_1_3.dtd">
+<!DOCTYPE module PUBLIC "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN" "https://checkstyle.org/dtds/configuration_1_3.dtd">
 
 <!--
     This configuration file was written by the eclipse-cs plugin configuration editor
@@ -14,7 +14,7 @@
     <module name="SuppressWarningsHolder"/>
     <module name="ConstantName"/>
     <module name="JavadocMethod">
-      <property name="scope" value="protected"/>
+      <property name="accessModifiers" value="protected, public"/>
     </module>
     <module name="JavadocType">
       <property name="scope" value="protected"/>
@@ -68,9 +68,7 @@
     <module name="EmptyStatement"/>
     <module name="EqualsHashCode"/>
     <module name="IllegalInstantiation"/>
-    <module name="MagicNumber">
-      <property name="constantWaiverParentToken" value="TYPECAST,METHOD_CALL,EXPR,ARRAY_INIT,UNARY_MINUS,UNARY_PLUS,ELIST,STAR,ASSIGN,PLUS,MINUS,DIV,LITERAL_NEW"/>
-    </module>
+    <module name="MagicNumber"/>
     <module name="MissingSwitchDefault"/>
     <module name="SimplifyBooleanExpression"/>
     <module name="SimplifyBooleanReturn"/>
@@ -90,6 +88,9 @@
     <module name="MissingJavadocType">
       <property name="scope" value="package"/>
     </module>
+    <module name="JavadocMethod">
+      <property name="accessModifiers" value="private"/>
+    </module>
   </module>
   <module name="SuppressWarningsFilter"/>
   <module name="Translation"/>