Skip to content

Change composer requirements to point to released package - #5860

Merged
leahecole merged 5 commits into
masterfrom
package-dag-test-utils
May 21, 2021
Merged

Change composer requirements to point to released package#5860
leahecole merged 5 commits into
masterfrom
package-dag-test-utils

Conversation

@leahecole

@leahecoleleahecole commented May 20, 2021

Copy link
Copy Markdown
Collaborator

Description

Successor to #5848 - if the tests are working, that means I did it right (it means the package is there!) The only type of failure that would happen if something in this process went wrong would be an import error.

Note: It's a good idea to open an issue first for discussion.

Checklist

@leahecole
leahecole requested a review from a team as a code ownerMay 20, 2021 22:06
@snippet-bot

Copy link
Copy Markdown

No region tags are edited in this PR.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@product-auto-labelproduct-auto-labelBot added the samples Issues that are directly related to samples. label May 20, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label May 20, 2021
@leahecoleleahecole changed the title Package dag test utilsChange composer requirements to point to released packageMay 20, 2021
@leahecoleleahecole added the api: composer Issues related to the Managed Service for Apache Airflow API. label May 20, 2021
@leahecoleleahecole mentioned this pull request May 21, 2021
8 tasks
@leahecole
leahecole merged commit 97d51f4 into masterMay 21, 2021
@leahecole
leahecole deleted the package-dag-test-utils branch May 21, 2021 14:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: composerIssues related to the Managed Service for Apache Airflow API.cla: yesThis human has signed the Contributor License Agreement.samplesIssues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@leahecole@busunkim96@rachael-ds@tmatsuo