Skip to content

chore: Removed unecessary pathlib param from test_deploy_dag#345

Merged
adlersantos merged 1 commit into
GoogleCloudPlatform:mainfrom
arjunsgill:get_composer_bucket_test
May 2, 2022
Merged

chore: Removed unecessary pathlib param from test_deploy_dag#345
adlersantos merged 1 commit into
GoogleCloudPlatform:mainfrom
arjunsgill:get_composer_bucket_test

Conversation

@arjunsgill

Copy link
Copy Markdown
Contributor

Description

Note: If you are adding or editing a dataset, please specify the dataset folder involved, e.g. datasets/google_trends

Checklist

Note: If an item applies to you, all of its sub-items must be fulfilled

  • (Required) This pull request is appropriately labeled
  • Please merge this pull request after it's approved
  • I'm adding or editing a feature
    • I have updated the README accordingly
    • I have added tests for the feature
  • I'm adding or editing a dataset
    • The Google Cloud Datasets team is aware of the proposed dataset
    • I put all my code inside datasets/<DATASET_NAME> and nothing outside of that directory
  • I'm adding/editing documentation
  • I'm submitting a bugfix
    • I have added tests to my bugfix (see the tests folder)
  • I'm refactoring or cleaning up some code

@arjunsgill arjunsgill requested a review from adlersantos May 2, 2022 13:57
@adlersantos adlersantos changed the title fix: removed unecessary pathlib param from test_deploy_dag chore: Removed unecessary pathlib param from test_deploy_dag May 2, 2022
@adlersantos adlersantos merged commit 45dd0b2 into GoogleCloudPlatform:main May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants