Skip to content

Commit 158c5fe

Browse files
committed
Renamed Helper to something meaningful and fixed condition
1 parent 14c0d5a commit 158c5fe

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,25 +17,25 @@
1717
package org.springframework.sbm.boot.upgrade_27_30.report.helper;
1818

1919
import org.springframework.sbm.boot.upgrade_27_30.report.SpringBootUpgradeReportSection;
20+
import org.springframework.sbm.build.migration.conditions.NoPluginRepositoryExistsCondition;
21+
import org.springframework.sbm.build.migration.conditions.NoRepositoryExistsCondition;
2022
import org.springframework.sbm.engine.context.ProjectContext;
21-
import org.springframework.sbm.properties.api.PropertiesSource;
2223

23-
import java.util.List;
2424
import java.util.Map;
2525

26-
public class MissingHelper implements SpringBootUpgradeReportSection.Helper<String>{
26+
public class AddSpringBootRepositoriesHelper implements SpringBootUpgradeReportSection.Helper<String>{
2727
@Override
2828
public String getDescription() {
2929
return null;
3030
}
3131

3232
@Override
3333
public boolean evaluate(ProjectContext context) {
34-
return false;
34+
return new NoRepositoryExistsCondition().evaluate(context) && new NoPluginRepositoryExistsCondition().evaluate(context);
3535
}
3636

3737
@Override
3838
public Map<String, String> getData() {
39-
return null;
39+
return Map.of();
4040
}
4141
}

components/sbm-recipes-boot-upgrade/src/main/resources/recipes/sbu30-report.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@
8686
8787
sections:
8888
- title: Add Spring Boot dependency and plugin milestone repository
89-
helper: org.springframework.sbm.boot.upgrade_27_30.report.helper.MissingHelper
89+
helper: org.springframework.sbm.boot.upgrade_27_30.report.helper.AddSpringBootRepositoriesHelper
9090
change: |-
9191
As currently only milestone releases exist these are required
9292
affected: |-

0 commit comments

Comments
 (0)