GoogleCloudPlatform / GoogleCloudPlatform/cloud-opensource-java

LinkageCheckRequest and LinkageCheckRequest.Builder

Abierto
#1,329 0 comentarios 0 reacciones 1 asignado Reclamado por @suztomo Ver en GitHub
p3 Tech Debt
Lenguaje dominante
Java
Estrellas
163
Forks
80
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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());
```

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.