Skip to content

Conversation

@keelerm84
Copy link
Member

@keelerm84 keelerm84 commented Oct 30, 2025

fixes #364


Note

Refines Files.new_data_source typing and adds corresponding imports, with no runtime logic changes.

  • Types:
    • Narrow Files.new_data_source return type to Optional[Callable[[Config, FeatureStore, Event], UpdateProcessor]].
    • Add required imports: Event, Callable, Config, FeatureStore, UpdateProcessor.

Written by Cursor Bugbot for commit d494584. This will update automatically on new commits. Configure here.

@keelerm84 keelerm84 requested a review from a team as a code owner October 30, 2025 13:37
@keelerm84 keelerm84 force-pushed the mk/sdk-1526/typehints branch from 76a5d03 to d494584 Compare October 30, 2025 13:45
@keelerm84 keelerm84 merged commit 52a7499 into main Oct 30, 2025
14 checks passed
@keelerm84 keelerm84 deleted the mk/sdk-1526/typehints branch October 30, 2025 18:55
keelerm84 pushed a commit that referenced this pull request Oct 30, 2025
🤖 I have created a release *beep* *boop*
---


##
[9.12.3](9.12.2...9.12.3)
(2025-10-30)


### Bug Fixes

* Fix overly generic type hint on File data source
([#365](#365))
([52a7499](52a7499)),
closes
[#364](#364)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Release 9.12.3 with a bug fix note and synchronized version updates
across files.
> 
> - **Release 9.12.3**:
> - Update versions in `.release-please-manifest.json`,
`ldclient/version.py`, and `pyproject.toml` to `9.12.3`.
>   - Update `PROVENANCE.md` example `SDK_VERSION` to `9.12.3`.
> - Add CHANGELOG entry for 9.12.3 noting bug fix: fix overly generic
type hint on File data source (`CHANGELOG.md`).
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
f392afa. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

update_processor_class has wrong types

3 participants