GoogleCloudPlatform / GoogleCloudPlatform/cloud-opensource-java

LinkageCheckRequest and LinkageCheckRequest.Builder

Aberta
#1,329 0 comentários 0 reações 1 responsável Reivindicada por @suztomo Ver no GitHub
p3 Tech Debt
Linguagem predominante
Java
Estrelas
163
Forks
80
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

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

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.