ci: Add disallow-legacy-apiary presubmit check to basic-pr-checks - #18639
ci: Add disallow-legacy-apiary presubmit check to basic-pr-checks#18639jcromanu wants to merge 1 commit into
Conversation
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @rileykarson, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit fb04281: Diff reportYour PR hasn't generated any diffs, but I'll let you know if a future commit does. |
fb04281 to
30cc11a
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 30cc11a: Diff reportYour PR hasn't generated any diffs, but I'll let you know if a future commit does. |
30cc11a to
4f5d4f5
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 4f5d4f5: Diff reportYour PR hasn't generated any diffs, but I'll let you know if a future commit does. |
rileykarson
left a comment
There was a problem hiding this comment.
Let me know when this is ready for review!
|
@rileykarson , the script is ready , should we wait until the last tickets of the apiary migration are done before merging it? |
Summary
Add an automated
disallow-legacy-apiarypresubmit job to.github/workflows/basic-pr-checks.ymlto prevent PRs from introducing new imports of the legacy Apiary client (google.golang.org/api/compute/*) or calls to deprecated client factories in handwritten overrides and templates.Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.