GoogleCloudPlatform / GoogleCloudPlatform/cloud-opensource-java

LinkageCheckRequest and LinkageCheckRequest.Builder

Aperta
#1,329 0 commenti 0 reazioni 1 assegnatario Rivendicata da @suztomo Vedi su GitHub
p3 Tech Debt
Lingua principale
Java
Stelle
163
Fork
80
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Current design of creating LinkageChecker for every class path looks strange. Let's change the parameters into `LinkageCheckRequest` class.

```
LinkageChecker linkageChecker = LinkageChecker.create();
LinkageCheckRequest.Builder request = LinkageCheckRequest.builder()
.classPath(classPath)
.exclusionFile(exclusionFile);

// current findSymbolProblems()
ImmutableSetMultimap = linkageChecker.check(request.build());
```

The `linkageChecker` instance does not hold any information. Maybe a static method should suffice:

```
LinkageCheckRequest.Builder request = LinkageCheckRequest.builder()
.classPath(classPath)
.exclusionFile(exclusionFile);
ImmutableSetMultimap result = LinkageChecker.check(request.build());
```

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.