Start off with a minimize public API

openrewrite · Dec 6, 2024 · 6c62959 · 6c62959
1 parent 576f8d5
commit 6c62959
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/rewrite-maven/src/main/java/org/openrewrite/maven/Assertions.java b/rewrite-maven/src/main/java/org/openrewrite/maven/Assertions.java
@@ -40,7 +40,7 @@ static void customizeExecutionContext(ExecutionContext ctx) {
             MavenExecutionContextView mctx = MavenExecutionContextView.view(ctx);
             MavenSettings settings = MavenSettings.readMavenSettingsFromDisk(mctx);
             if (settings != null) {
-                settings.updatePasswords(mctx);
+                settings.maybeDecryptPasswords(mctx);
             }
             mctx.setMavenSettings(settings);
         }

diff --git a/rewrite-maven/src/main/java/org/openrewrite/maven/MavenSecuritySettings.java b/rewrite-maven/src/main/java/org/openrewrite/maven/MavenSecuritySettings.java
@@ -44,7 +44,7 @@ public class MavenSecuritySettings {
     @Nullable
     String master;
 
-    public static @Nullable MavenSecuritySettings parse(Parser.Input source, ExecutionContext ctx) {
+    private static @Nullable MavenSecuritySettings parse(Parser.Input source, ExecutionContext ctx) {
         try {
             return new Interpolator().interpolate(
                     MavenXmlMapper.readMapper().readValue(source.getSource(ctx), MavenSecuritySettings.class));
@@ -54,7 +54,7 @@ public class MavenSecuritySettings {
         }
     }
 
-    public static @Nullable MavenSecuritySettings parse(Path settingsPath, ExecutionContext ctx) {
+    private static @Nullable MavenSecuritySettings parse(Path settingsPath, ExecutionContext ctx) {
         return parse(new Parser.Input(settingsPath, () -> {
             try {
                 return Files.newInputStream(settingsPath);
@@ -98,7 +98,7 @@ private static boolean exists(Path path) {
         }
     }
 
-    public MavenSecuritySettings merge(@Nullable MavenSecuritySettings installSettings) {
+    private MavenSecuritySettings merge(@Nullable MavenSecuritySettings installSettings) {
         return installSettings == null ? this : new MavenSecuritySettings(
             master == null ? installSettings.master : master
         );

diff --git a/rewrite-maven/src/main/java/org/openrewrite/maven/MavenSettings.java b/rewrite-maven/src/main/java/org/openrewrite/maven/MavenSettings.java
@@ -126,7 +126,7 @@ public MavenSettings(@Nullable String localRepository, @Nullable Profiles profil
                 .orElse(installSettings);
     }
 
-    void updatePasswords(ExecutionContext ctx) {
+    void maybeDecryptPasswords(ExecutionContext ctx) {
         MavenSecuritySettings security = MavenSecuritySettings.readMavenSecuritySettingsFromDisk(ctx);
         if (security == null) {
             return;