Skip to content

🎉 Source Facebook-Marketing: Use latest CDK version possible - #40645

Merged
Baz (bazarnov) merged 6 commits into
masterfrom
baz/source/fb-marketing/use-latest-cdk
Jul 4, 2024
Merged

Baz (bazarnov) merged 6 commits into
masterfrom
baz/source/fb-marketing/use-latest-cdk

Conversation

@bazarnov

Copy link
Copy Markdown
Contributor

What

Resolving:

How

  • updated spec.py to import references from pydantic.v1 to preserve the structure of the spec
  • bumped the version

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

@bazarnov Baz (bazarnov) self-assigned this Jul 1, 2024
@vercel

vercel Bot commented Jul 1, 2024 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Jul 4, 2024 5:35pm

@bazarnov
Baz (bazarnov) marked this pull request as ready for review July 1, 2024 12:36
@bazarnov
Baz (bazarnov) requested a review from a team July 1, 2024 12:36
@octavia-squidington-iii Octavia Squidington III (octavia-squidington-iii) added the area/documentation Improvements or additions to documentation label Jul 1, 2024
@bazarnov
Baz (bazarnov) requested a review from a team July 1, 2024 12:39
@bazarnov

Copy link
Copy Markdown
Contributor Author

Maxime Carbonneau-Leclerc (@maxi297) The logic of the source didn't change, but some of the unit tests are now failing after upgrading to the CDK == ^2. Could you please check them once you have a chance, please?

@maxi297

Copy link
Copy Markdown
Contributor

Note that there might be this too to take into account

@bazarnov

Baz (bazarnov) commented Jul 4, 2024 •

Copy link
Copy Markdown
Contributor Author

Note that there might be this too to take into account

I'm not sure I understood your point or where this is used within Facebook-Marketing's integration tests.` Could you please specify how this impacts the integration tests?

@bazarnov
Baz (bazarnov) enabled auto-merge (squash) July 4, 2024 17:35
@bazarnov
Baz (bazarnov) merged commit f43ec31 into master Jul 4, 2024
@bazarnov
Baz (bazarnov) deleted the baz/source/fb-marketing/use-latest-cdk branch July 4, 2024 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/facebook-marketing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants