Skip to content

Commit 05ca325

Browse files
committed
[CI] Replace HashMap with Map to make code more generic
1 parent 5adaf2e commit 05ca325

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

.github/quarkus-ecosystem-issue.java

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
import java.io.UncheckedIOException;
4040
import java.util.HashMap;
4141
import java.util.List;
42+
import java.util.Map;
4243
import java.util.regex.Matcher;
4344
import java.util.regex.Pattern;
4445
import java.io.BufferedReader;
@@ -90,11 +91,11 @@ public void run() {
9091
// name prefixes. For each job prefix we keep two issues, one for
9192
// quarkus integration tests and the other for mandrel integration
9293
// tests.
93-
final HashMap<GHIssue, String> issues = new HashMap<>();
94-
final HashMap<GHIssue, String> mandrelITIssues = new HashMap<>();
94+
final Map<GHIssue, String> issues = new HashMap<>();
95+
final Map<GHIssue, String> mandrelITIssues = new HashMap<>();
9596
// We use two more HashMaps to associate issues with known job failures
96-
final HashMap<GHIssue, List<GHWorkflowJob>> failedMandrelJobs = new HashMap<>();
97-
final HashMap<GHIssue, List<GHWorkflowJob>> failedMandrelITJobs = new HashMap<>();
97+
final Map<GHIssue, List<GHWorkflowJob>> failedMandrelJobs = new HashMap<>();
98+
final Map<GHIssue, List<GHWorkflowJob>> failedMandrelITJobs = new HashMap<>();
9899

99100
// Get the github issue number and repository from the logs
100101
//
@@ -153,7 +154,7 @@ public void run() {
153154
}
154155

155156

156-
private void recordFailedJob(final HashMap<GHIssue, List<GHWorkflowJob>> failedJobs, HashMap<GHIssue, String> issues, GHWorkflowJob job) {
157+
private void recordFailedJob(final Map<GHIssue, List<GHWorkflowJob>> failedJobs, Map<GHIssue, String> issues, GHWorkflowJob job) {
157158
for (GHIssue issue: issues.keySet()) {
158159
if (job.getName().startsWith(issues.get(issue))) {
159160
List<GHWorkflowJob> failedJobsList = failedJobs.get(issue);
@@ -205,8 +206,8 @@ private void reportFailedJobs(GHIssue issue, List<GHWorkflowJob> failedJobs) thr
205206
}
206207

207208

208-
private void processLogs(GitHub github, GHWorkflowJob job, HashMap<GHIssue, String> issues, HashMap<GHIssue, String> mandrelITIssues,
209-
TriConsumer<GHIssue, GHWorkflowJob, HashMap<GHIssue, String>> process, String... filters) {
209+
private void processLogs(GitHub github, GHWorkflowJob job, Map<GHIssue, String> issues, Map<GHIssue, String> mandrelITIssues,
210+
TriConsumer<GHIssue, GHWorkflowJob, Map<GHIssue, String>> process, String... filters) {
210211
String fullContent = getJobsLogs(job, filters);
211212
if (fullContent.isEmpty()) {
212213
return;
@@ -242,7 +243,7 @@ private void processLogs(GitHub github, GHWorkflowJob job, HashMap<GHIssue, Stri
242243
}
243244
}
244245

245-
private void processITJobs(GHIssue issue, GHWorkflowJob job, HashMap<GHIssue, String> issues) {
246+
private void processITJobs(GHIssue issue, GHWorkflowJob job, Map<GHIssue, String> issues) {
246247
if (issue == null) {
247248
System.out.println(String.format("Unable to find the issue %s in project %s", issue.getNumber(), issue.getRepository().getName()));
248249
System.exit(-1);
@@ -256,7 +257,7 @@ private void processITJobs(GHIssue issue, GHWorkflowJob job, HashMap<GHIssue, St
256257
}
257258
}
258259

259-
private void processSyncJobs(GHIssue issue, GHWorkflowJob job, HashMap<GHIssue, String> issues) {
260+
private void processSyncJobs(GHIssue issue, GHWorkflowJob job, Map<GHIssue, String> issues) {
260261
try {
261262
if (issue == null) {
262263
System.out.println(String.format("Unable to find the issue %s in project %s", issue.getNumber(), issue.getRepository().getName()));

0 commit comments

Comments
 (0)