Skip to content
This repository has been archived by the owner on Jan 25, 2022. It is now read-only.

Feature/fetch upstram #5

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Feature/fetch upstram #5

wants to merge 11 commits into from

Conversation

andrea-t94
Copy link

Branch to solve some merge conflict by fetching upstream

luandy64 and others added 10 commits June 28, 2021 10:13
* Update PK for `ads_insights_country`

* Update test expectations
* Bump to v1.15.0, update changelog

* Fix changelog headers
* Update streams list in README; fix typo

* Remove hourly audience breakdown

Co-authored-by: Andy Lu <[email protected]>
* Bump facebook_business to 11.0.0

* Get on the latest sdk version
* Reorganize config file

* Fix missing key

* Make ensure_env run first

* Run unit tests and pylint in parallel

* Add checkouts to every job

* Name steps, add parallelism to Unit Tests

* Only run if you have tests

* Parallelize integration tests; serialize unit tests

* Acutally run integration tests

* Fix copy pasta

* Add logging around what tests are running

* Fix typo

* Remove unused code

* Add slack notifications for failures

* Reorder work to shrink the diff

* Add build_daily back in

* Add the version line back in

* Update docker image

* Fix run command for the newer docker image

* Decouple tests without parallelism

* Fix build_daily too

* Fix path to tests

* fail if we are missing a tap-tester test in the circle-config

* reuse commit workflow as build-daily

* fix typo

* Add build job for github status

* Add missing context

* Refer to names of other jobs instead of the template?

* Remove yaml reuse

* Add deploy step to commit workflow, list jobs for build_daily workflow

* Add filter to only deploy from master

* Fix copy pasta

* Add notifications on success and on failure

Co-authored-by: kspeer <[email protected]>
@andrea-t94 andrea-t94 requested a review from mavimo October 6, 2021 14:25
@andrea-t94 andrea-t94 self-assigned this Oct 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants