Skip to content

Conversation

@candiduslynx
Copy link
Contributor

Inspired by the diff in #10831

@candiduslynx candiduslynx requested a review from hermanschaaf May 19, 2023 13:13
@hermanschaaf
Copy link
Member

Nice, let's just hang back with this a little please, I think we're going to need to make some changes to the Arrow doc format first and I'm not sure how that's going to look yet.

@hermanschaaf
Copy link
Member

Related PR: cloudquery/plugin-sdk#896

@candiduslynx
Copy link
Contributor Author

@hermanschaaf I've updates the SDK to v3.6.3, that includes cloudquery/plugin-sdk#896.
I presume now this code is OK to be merged and leaves us with a betted diff for src migration

@candiduslynx candiduslynx added the automerge Automatically merge once required checks pass label May 24, 2023
@kodiakhq kodiakhq bot merged commit 03e2d71 into main May 24, 2023
@kodiakhq kodiakhq bot deleted the feat/changes-diff branch May 24, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants