Skip to content

[ggj][engx] fix: make service.yaml optional #480

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
miraleung merged 11 commits into from
Nov 12, 2020
Merged

[ggj][engx] fix: make service.yaml optional #480

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
0725750
fix: swap assertEquals args in JavaWriterVisitorTest to match (expect…
miraleung Nov 7, 2020
67f3a15
fix: swap assertEquals args in ImportWriterVisitorTest to match (expe…
miraleung Nov 7, 2020
1e1a86b
fix: add node validator to refactor/centralize null element checks
miraleung Nov 7, 2020
0a6662c
test: update logging integration goldens with gapic.yaml
miraleung Nov 10, 2020
e9cdf58
feat!: remove gapic.yaml from Bazel and input interface
miraleung Nov 10, 2020
827049d
feat!: remove all per-RPC batching codegen logic
miraleung Nov 10, 2020
ea153c5
fix: update tests
miraleung Nov 10, 2020
7f15ea5
Merge branch 'gp/g1' of github.com:googleapis/gapic-generator-java in…
miraleung Nov 10, 2020
0722246
fix: handle wildcard-only resource_reference
miraleung Nov 11, 2020
8087ace
fix: make service.yaml optional
miraleung Nov 12, 2020
58f9571
Merge branch 'master' into gp/g4-test-iam
miraleung Nov 12, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/main/java/com/google/api/generator/gapic/protoparser/Parser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ public class Parser {
private static final String COLON = ":";
private static final String DEFAULT_PORT = "443";
private static final String DOT = ".";

// Allow other parsers to access this.
protected static final SourceCodeInfoParser SOURCE_CODE_INFO_PARSER = new SourceCodeInfoParser();

Expand All @@ -83,7 +82,9 @@ public static GapicContext parse(CodeGeneratorRequest request) {
Optional<String> serviceYamlConfigPathOpt =
PluginArgumentParser.parseServiceYamlConfigPath(request);
Optional<com.google.api.Service> serviceYamlProtoOpt =
ServiceYamlParser.parse(serviceYamlConfigPathOpt.get());
serviceYamlConfigPathOpt.isPresent()
? ServiceYamlParser.parse(serviceYamlConfigPathOpt.get())
: Optional.empty();

// Keep message and resource name parsing separate for cleaner logic.
// While this takes an extra pass through the protobufs, the extra time is relatively trivial
Expand Down