Skip to content

chore: prepare ServiceInfo base class RecordUpdateListener for cython#1263

Merged
bdraco merged 2 commits into
masterfrom
with_record_update_listener
Sep 10, 2023
Merged

chore: prepare ServiceInfo base class RecordUpdateListener for cython#1263
bdraco merged 2 commits into
masterfrom
with_record_update_listener

Conversation

@bdraco

@bdraco bdraco commented Sep 10, 2023

Copy link
Copy Markdown
Member

This should effectively do nothing except make ServiceInfo cythonable since cython requires the base class to be a cdef

@bdraco bdraco changed the title feat: prepare ServiceInfo base class RecordUpdateListener for cython chore: prepare ServiceInfo base class RecordUpdateListener for cython Sep 10, 2023
@codecov

codecov Bot commented Sep 10, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage is 100.00% of modified lines.

Files Changed Coverage
src/zeroconf/__init__.py 100.00%
src/zeroconf/_engine.py 100.00%
src/zeroconf/_handlers/record_manager.py 100.00%
src/zeroconf/_record_update.py 100.00%
src/zeroconf/_services/browser.py 100.00%
src/zeroconf/_services/info.py 100.00%
src/zeroconf/_updates.py 100.00%

📢 Thoughts on this report? Let us know!.

@bdraco bdraco marked this pull request as ready for review September 10, 2023 22:05
@bdraco bdraco merged commit 6bf5d95 into master Sep 10, 2023
@bdraco bdraco deleted the with_record_update_listener branch September 10, 2023 22:05
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.

1 participant