edx-analytics-pipeline icon indicating copy to clipboard operation
edx-analytics-pipeline copied to clipboard

Published 20 hours ago • verified publisher icon openedx

Removed all BigQuery related code.

Open HassanJaveed84 opened this issue 4 years ago • 3 comments

Analytics Pipeline Pull Request

Make sure that the following steps are done before merging:

  • [ ] If you have a migration please contact data engineering team before merging.
  • [ ] Before merging run full acceptance tests suite and provide URL for the acceptance tests run.
  • [ ] A member of data engineering team has approved the pull request.

HassanJaveed84 avatar Jun 25 '20 14:06 HassanJaveed84

Additional A/C: Address feedback noted here: https://github.com/edx/edx-analytics-pipeline/pull/824/files#r452311767

HassanJaveed84 avatar Jul 09 '20 20:07 HassanJaveed84

Additional A/C: Address feedback noted here: https://github.com/edx/edx-analytics-pipeline/pull/824/files#r452311767

I'm intentionally ignoring this code cleanliness improvement as we're deprecating this codebase soon anyway.

pwnage101 avatar Oct 30 '20 14:10 pwnage101

I'm giving up on this because I can't get isort to pass.

pwnage101 avatar Oct 30 '20 20:10 pwnage101