Skip to content

refactor(gcp-codegen): Make ServiceDNS optional#5956

Merged
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
erezrokah:refactor/make_dns_name_optional
Dec 28, 2022
Merged

refactor(gcp-codegen): Make ServiceDNS optional#5956
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
erezrokah:refactor/make_dns_name_optional

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

Follow up to #5534.

In most cases ServiceDNS can be inferred from Service (not sure but we might want to update Service when it doesn't)

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 26, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cloudquery-web 🔄 Building (Inspect) Dec 26, 2022 at 9:46AM (UTC)

@erezrokah erezrokah requested a review from bbernays December 26, 2022 09:46
@cq-bot cq-bot added the gcp label Dec 26, 2022
@erezrokah erezrokah changed the title refactor: Make ServiceDNS optional refactor(gcp-codegen): Make ServiceDNS optional Dec 26, 2022
@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Dec 26, 2022
@kodiakhq kodiakhq bot merged commit 84dd412 into cloudquery:main Dec 28, 2022
@erezrokah erezrokah deleted the refactor/make_dns_name_optional branch December 28, 2022 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants