Skip to content

Commit 26f4340

Browse files
committed
review up
1 parent fcd2225 commit 26f4340

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

tasks/git/src/main/java/com/walmartlabs/concord/plugins/git/client/GitHubClient.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,8 @@
4040
import java.util.*;
4141
import java.util.regex.Pattern;
4242

43+
import static java.util.Objects.requireNonNull;
44+
4345
public class GitHubClient {
4446

4547
private final static Logger log = LoggerFactory.getLogger(GitHubClient.class);
@@ -76,8 +78,8 @@ public Map<String, Object> singleObjectResult(String method, String path, Object
7678
public void forEachPage(String path, Map<String, String> params, int pageSize, PageHandler handler)
7779
throws IOException, InterruptedException, URISyntaxException {
7880

79-
Objects.requireNonNull(path, "'path' must not be null");
80-
Objects.requireNonNull(params, "'params' must not be null");
81+
requireNonNull(path, "'path' must not be null");
82+
requireNonNull(params, "'params' must not be null");
8183

8284
var effectiveParams = new HashMap<>(params);
8385
effectiveParams.put("per_page", String.valueOf(pageSize));
@@ -109,7 +111,7 @@ private static String buildApiUrl(String baseUrl, String path) throws URISyntaxE
109111
prefix = "/api/v3";
110112
}
111113

112-
var scheme = Objects.requireNonNull(uri.getScheme(), "Base URL without schema");
114+
var scheme = requireNonNull(uri.getScheme(), "Base URL without schema");
113115
var port = uri.getPort();
114116

115117
var apiUri = new URI(scheme, null, host, port, joinPaths(prefix, path), null, null);

0 commit comments

Comments
 (0)