make access modifier as restrictive as possible

skippy-io · Nov 26, 2023 · b5bf841 · b5bf841
1 parent 145f717
commit b5bf841
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 10 deletions.
diff --git a/skippy-core/src/main/java/io/skippy/core/AnalyzedFile.java b/skippy-core/src/main/java/io/skippy/core/AnalyzedFile.java
@@ -23,7 +23,7 @@
  *     <li>MD5 hash of class file</li>
  * </ul>
  */
-public class AnalyzedFile {
+class AnalyzedFile {
 
     private static final Logger LOGGER = LogManager.getLogger(AnalyzedFile.class);
 

diff --git a/skippy-core/src/main/java/io/skippy/core/FullyQualifiedClassName.java b/skippy-core/src/main/java/io/skippy/core/FullyQualifiedClassName.java
@@ -4,4 +4,4 @@
  * Allows for more meaningful typing, e.g., {@code Map<FullyQualifiedClassName, List<FullyQualifiedClassName>>}
  * instead of {@code Map<String, List<String>>}.
  */
-public record FullyQualifiedClassName(String fullyQualifiedClassName) {}
+record FullyQualifiedClassName(String fullyQualifiedClassName) {}
diff --git a/skippy-core/src/main/java/io/skippy/core/SkippyConstants.java b/skippy-core/src/main/java/io/skippy/core/SkippyConstants.java
@@ -5,16 +5,16 @@
 /**
  * Comment to make the JavaDoc task happy.
  */
-public class SkippyConstants {
+class SkippyConstants {
 
     /**
      * The directory that contains the Skippy analysis.
      */
-    public static final Path SKIPPY_DIRECTORY = Path.of("skippy");
+    static final Path SKIPPY_DIRECTORY = Path.of("skippy");
 
     /**
      * The file that contains data for all {@link AnalyzedFile}s.
      */
-    public static final Path SKIPPY_ANALYSIS_FILE = Path.of("analyzedFiles.txt");
+    static final Path SKIPPY_ANALYSIS_FILE = Path.of("analyzedFiles.txt");
 
 }
diff --git a/skippy-core/src/main/java/io/skippy/core/TestImpactAnalysis.java b/skippy-core/src/main/java/io/skippy/core/TestImpactAnalysis.java
@@ -18,14 +18,11 @@
 /**
  * A mapping between tests and the classes they cover.
  */
-public class TestImpactAnalysis {
+class TestImpactAnalysis {
 
     private static final Logger LOGGER = LogManager.getLogger(TestImpactAnalysis.class);
 
-    /**
-     * Indicates that no test impact analysis was found.
-     */
-    public static final TestImpactAnalysis UNAVAILABLE = new TestImpactAnalysis(emptyMap());
+    static final TestImpactAnalysis UNAVAILABLE = new TestImpactAnalysis(emptyMap());
 
     private final Map<FullyQualifiedClassName, List<FullyQualifiedClassName>> testCoverage;