googleapis / python-bigtable

Apache License 2.0
68 stars 59 forks source link

chore: add cross_sync annotations #1000

Closed daniel-sanche closed 4 days ago

daniel-sanche commented 4 months ago

PR 2/x for adding the sync surface for the new data client

This PR restructures the existing async library to better support the sync side. Specifically:

Currently, we need to run the conversion process manually (using python .cross_sync/generate.py, and the sync files are not included in the codebase. The next PR will add the sync artifacts, and add the generation pipeline into the CI system

daniel-sanche commented 1 week ago

Is there a test to validate that the sync surface is re-generated when there's change to async code?

Yes, but that's in the next PR, since we don't have the generated files here yet