Skip to content

feat: Onboarding TheLook E-Commerce dataset#294

Merged
adlersantos merged 22 commits into
GoogleCloudPlatform:mainfrom
alick-at-google:alick-thelook-ecommerce
Feb 28, 2022
Merged

feat: Onboarding TheLook E-Commerce dataset#294
adlersantos merged 22 commits into
GoogleCloudPlatform:mainfrom
alick-at-google:alick-thelook-ecommerce

Conversation

@alick-at-google

@alick-at-google alick-at-google commented Feb 11, 2022

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

@adlersantos adlersantos added the data onboarding Onboard a dataset or submit a pipeline label Feb 11, 2022
@leahecole leahecole requested review from leahecole and removed request for leahecole February 11, 2022 20:51
Comment thread datasets/thelook_ecommerce/pipelines/thelook_ecommerce/pipeline.yaml Outdated
Comment thread datasets/thelook_ecommerce/pipelines/dataset.yaml Outdated
Comment thread datasets/thelook_ecommerce/pipelines/thelook_ecommerce/pipeline.yaml Outdated
@adlersantos adlersantos merged commit 15f663a into GoogleCloudPlatform:main Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data onboarding Onboard a dataset or submit a pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants