fix(concurrent_declarative_source): set `None` defaults for optional constructor args, remove unnecessary `Generic` implementation#738

Merged

aaronsteers merged 11 commits intomainfrom

devin/1735434900-fix-optional-state-parameter

Aug 29, 2025

Commits on Aug 29, 2025

fix(concurrent_declarative_source): add default value to optional state parameter

devin-ai-integration[bot]
and
aaronsteers
committed
Apply suggestion from @aaronsteers
aaronsteers
authored
style: use named parameters in ConcurrentDeclarativeSource constructor call

devin-ai-integration[bot]
and
aaronsteers
committed
fix: apply ruff 0.11.5 formatting to resolve CI format check discrepancy

devin-ai-integration[bot]
and
aaronsteers
committed
fix: add missing source_config parameter to ConcurrentDeclarativeSource constructor in test

devin-ai-integration[bot]
and
aaronsteers
committed
preserve positional order of constructor, force kwargs where necessary for correctness
aaronsteers
authored
explcit default to None for catalog and config
aaronsteers
authored
update create_source() to properly declare optional inputs
aaronsteers
authored
fix: correct return type annotation in get_config_and_catalog_from_args to resolve MyPy error

devin-ai-integration[bot]
and
aaronsteers
committed
Merge branch 'devin/1735434900-fix-optional-state-parameter' of https://git-manager.devin.ai/proxy/github.com/airbytehq/airbyte-python-cdk into devin/1735434900-fix-optional-state-parameter
devin-ai-integration[bot]
committed
fix: correct state parameter type annotation in create_source to match ConcurrentDeclarativeSource signature

devin-ai-integration[bot]
and
aaronsteers
committed