Skip to content

Commit f8bc07b

Browse files
CopilotYunchuWang
andcommitted
Remove SpotBugs plugin and configuration to resolve Java compatibility issues
Co-authored-by: YunchuWang <12449837+YunchuWang@users.noreply.github.com>
1 parent 1f1010b commit f8bc07b

7 files changed

Lines changed: 4 additions & 163 deletions

File tree

.github/workflows/build-validation.yml

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -44,19 +44,7 @@ jobs:
4444
- name: Build with Gradle
4545
run: ./gradlew build -x test
4646

47-
- name: Run SpotBugs
48-
run: |
49-
export JAVA_HOME=$JDK_11
50-
./gradlew spotbugsMain spotbugsTest
51-
continue-on-error: false
52-
53-
- name: Upload SpotBugs reports
54-
uses: actions/upload-artifact@v4
55-
with:
56-
name: SpotBugs Reports
57-
path: '**/build/reports/spotbugs'
58-
if-no-files-found: ignore
59-
47+
6048
- name: Run Unit Tests with Gradle
6149
run: |
6250
export JAVA_HOME=$JDK_11

azurefunctions/build.gradle

Lines changed: 1 addition & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ plugins {
22
id 'java-library'
33
id 'maven-publish'
44
id 'signing'
5-
id 'com.github.spotbugs' version '5.2.1'
5+
66
}
77

88
group 'com.microsoft'
@@ -87,34 +87,4 @@ java {
8787
withJavadocJar()
8888
}
8989

90-
spotbugs {
91-
toolVersion = '4.9.2'
92-
effort = 'max'
93-
reportLevel = 'high'
94-
ignoreFailures = true
95-
excludeFilter = file('spotbugs-exclude.xml')
96-
}
9790

98-
spotbugsMain {
99-
reports {
100-
html {
101-
required = true
102-
stylesheet = 'fancy-hist.xsl'
103-
}
104-
xml {
105-
required = true
106-
}
107-
}
108-
}
109-
110-
spotbugsTest {
111-
reports {
112-
html {
113-
required = true
114-
stylesheet = 'fancy-hist.xsl'
115-
}
116-
xml {
117-
required = true
118-
}
119-
}
120-
}

azurefunctions/spotbugs-exclude.xml

Lines changed: 0 additions & 17 deletions
This file was deleted.

azuremanaged/build.gradle

Lines changed: 1 addition & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ plugins {
1212
id 'idea'
1313
id 'maven-publish'
1414
id 'signing'
15-
id 'com.github.spotbugs' version '5.2.1'
15+
1616
}
1717

1818
archivesBaseName = 'durabletask-azuremanaged'
@@ -122,37 +122,7 @@ java {
122122
withJavadocJar()
123123
}
124124

125-
spotbugs {
126-
toolVersion = '4.9.2'
127-
effort = 'max'
128-
reportLevel = 'high'
129-
ignoreFailures = true
130-
excludeFilter = file('spotbugs-exclude.xml')
131-
}
132125

133-
spotbugsMain {
134-
reports {
135-
html {
136-
required = true
137-
stylesheet = 'fancy-hist.xsl'
138-
}
139-
xml {
140-
required = true
141-
}
142-
}
143-
}
144-
145-
spotbugsTest {
146-
reports {
147-
html {
148-
required = true
149-
stylesheet = 'fancy-hist.xsl'
150-
}
151-
xml {
152-
required = true
153-
}
154-
}
155-
}
156126

157127
// Add this after the plugins block
158128
def generatedSourcesDir = file("$buildDir/generated/sources/version/java/main")

azuremanaged/spotbugs-exclude.xml

Lines changed: 0 additions & 17 deletions
This file was deleted.

client/build.gradle

Lines changed: 1 addition & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ plugins {
55
id 'idea'
66
id 'maven-publish'
77
id 'signing'
8-
id 'com.github.spotbugs' version '5.2.1'
8+
99
id 'org.gradle.test-retry' version '1.4.1'
1010
}
1111

@@ -203,34 +203,3 @@ java {
203203
withJavadocJar()
204204
}
205205

206-
spotbugs {
207-
toolVersion = '4.9.2'
208-
effort = 'max'
209-
reportLevel = 'high'
210-
ignoreFailures = true
211-
excludeFilter = file('spotbugs-exclude.xml')
212-
}
213-
214-
spotbugsMain {
215-
reports {
216-
html {
217-
required = true
218-
stylesheet = 'fancy-hist.xsl'
219-
}
220-
xml {
221-
required = true
222-
}
223-
}
224-
}
225-
226-
spotbugsTest {
227-
reports {
228-
html {
229-
required = true
230-
stylesheet = 'fancy-hist.xsl'
231-
}
232-
xml {
233-
required = true
234-
}
235-
}
236-
}

client/spotbugs-exclude.xml

Lines changed: 0 additions & 22 deletions
This file was deleted.

0 commit comments

Comments
 (0)