Skip to content

Commit ac7dc77

Browse files
authored
Merge pull request #135 from nebula-plugins/address-is-boolean-deprecation
Replace isStrictMode to getStrictMode to address Declaring an 'is-' property with a Boolean type has been deprecated.
2 parents 6ff5144 + 8f2b2dc commit ac7dc77

File tree

3 files changed

+12
-7
lines changed

3 files changed

+12
-7
lines changed

.github/workflows/nebula.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,14 @@ jobs:
3838
8
3939
${{ matrix.java }}
4040
java-package: jdk
41-
- uses: actions/cache@v1
41+
- uses: actions/cache@v3
4242
id: gradle-cache
4343
with:
4444
path: ~/.gradle/caches
4545
key: ${{ runner.os }}-gradle-${{ hashFiles('**/gradle/dependency-locks/*.lockfile') }}
4646
restore-keys: |
4747
- ${{ runner.os }}-gradle-
48-
- uses: actions/cache@v1
48+
- uses: actions/cache@v3
4949
id: gradle-wrapper-cache
5050
with:
5151
path: ~/.gradle/wrapper
@@ -78,14 +78,14 @@ jobs:
7878
8
7979
21
8080
java-package: jdk
81-
- uses: actions/cache@v1
81+
- uses: actions/cache@v3
8282
id: gradle-cache
8383
with:
8484
path: ~/.gradle/caches
8585
key: ${{ runner.os }}-gradle-${{ hashFiles('**/gradle/dependency-locks/*.lockfile') }}
8686
restore-keys: |
8787
- ${{ runner.os }}-gradle-
88-
- uses: actions/cache@v1
88+
- uses: actions/cache@v3
8989
id: gradle-wrapper-cache
9090
with:
9191
path: ~/.gradle/wrapper
@@ -123,14 +123,14 @@ jobs:
123123
8
124124
21
125125
java-package: jdk
126-
- uses: actions/cache@v1
126+
- uses: actions/cache@v3
127127
id: gradle-cache
128128
with:
129129
path: ~/.gradle/caches
130130
key: ${{ runner.os }}-gradle-${{ hashFiles('**/gradle/dependency-locks/*.lockfile') }}
131131
restore-keys: |
132132
- ${{ runner.os }}-gradle-
133-
- uses: actions/cache@v1
133+
- uses: actions/cache@v3
134134
id: gradle-wrapper-cache
135135
with:
136136
path: ~/.gradle/wrapper

src/main/groovy/netflix/nebula/dependency/recommender/DependencyRecommendationsPlugin.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ public void execute(DependencyResolveDetails details) {
129129
details.because("Recommending version " + version + " for dependency " + coordinate + " via " + strategyText + "\n" +
130130
"\twith reasons: " + StringUtils.join(getReasonsRecursive(project), ", "));
131131
} else {
132-
if (recommendationProviderContainer.isStrictMode()) {
132+
if (recommendationProviderContainer.getStrictMode()) {
133133
String errorMessage = "Dependency " + details.getRequested().getGroup() + ":" + details.getRequested().getName() + " omitted version with no recommended version. General causes include a dependency being removed from the recommendation source or not applying a recommendation source to a project that depends on another project using a recommender.";
134134
project.getLogger().error(errorMessage);
135135
throw new GradleException(errorMessage);

src/main/groovy/netflix/nebula/dependency/recommender/provider/RecommendationProviderContainer.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,12 @@ public void setStrategy(RecommendationStrategies strategy) {
222222
this.strategy = strategy;
223223
}
224224

225+
@Deprecated
225226
public Boolean isStrictMode() {
227+
return getStrictMode();
228+
}
229+
230+
public Boolean getStrictMode() {
226231
return strictMode;
227232
}
228233

0 commit comments

Comments
 (0)