Repository navigation
Conversation
|
Note 📝 PR Converted to Draft More info...Thank you for creating this PR. As a policy to protect our engineers' time, Airbyte requires all PRs to be created first in draft status. Your PR has been automatically converted to draft status in respect for this policy. As soon as your PR is ready for formal review, you can proceed to convert the PR to "ready for review" status by clicking the "Ready for review" button at the bottom of the PR page. To skip draft status in future PRs, please include |
👋 Welcome to PyAirbyte!Thank you for your contribution from HootRock/PyAirbyte! We're excited to have you in the Airbyte community. 💡 Show Tips and TricksTesting This PyAirbyte VersionYou can test this version of PyAirbyte using the following: # Run PyAirbyte CLI from this branch:
uvx --from 'git+https://github.com/HootRock/PyAirbyte.git@fix/respect-supported-sync-modes' pyairbyte --help
# Install PyAirbyte from this branch for development:
pip install 'git+https://github.com/HootRock/PyAirbyte.git@fix/respect-supported-sync-modes'PR Slash CommandsAs needed or by request, Airbyte Maintainers can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful ResourcesIf you have any questions, feel free to ask in the PR comments or join our community:
|
Fixes #917.
Source.get_configured_catalog()currently selectsincrementalby default even when a stream advertises onlyfull_refresh. Selectfull_refreshfor that case, while keeping the incremental preference for dual-mode streams and the existingforce_full_refreshbehavior from #899.Six regression cases construct real protocol catalogs through the public method and cover all supported capability combinations with and without forced refresh. Discovery is mocked; no connector or live provider is run.
Validation:
Broader offline unit run: 1269 passed, 1 skipped, 2 failures. Both failures reproduce on clean main: the Cloud workspace-scope test makes an unmocked external request (blocked here), and the time-format test assumes UTC on this UTC+8 host. Nine unrelated constants subprocess cases were excluded after a local subprocess timeout. External sockets/DNS were prohibited; no full-suite or live-integration pass is claimed.
reviewed and tested locally.