Skip to content

Ignore stale records when considering to ask questions#1014

Closed
bdraco wants to merge 1 commit into
python-zeroconf:masterfrom
bdraco:ignore_stale_records_asking
Closed

Ignore stale records when considering to ask questions#1014
bdraco wants to merge 1 commit into
python-zeroconf:masterfrom
bdraco:ignore_stale_records_asking

Conversation

@bdraco

@bdraco bdraco commented Oct 22, 2021

Copy link
Copy Markdown
Member

Fixes part 1 of #1013 (comment)

@codecov-commenter

codecov-commenter commented Oct 22, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1014 (7f2de64) into master (6a11f24) will decrease coverage by 0.03%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1014      +/-   ##
==========================================
- Coverage   99.84%   99.80%   -0.04%     
==========================================
  Files          23       23              
  Lines        2543     2544       +1     
  Branches      471      472       +1     
==========================================
  Hits         2539     2539              
  Misses          3        3              
- Partials        1        2       +1     
Impacted Files Coverage Δ
zeroconf/_protocol/outgoing.py 99.52% <75.00%> (-0.48%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a11f24...7f2de64. Read the comment docs.

@bdraco

bdraco commented Oct 22, 2021

Copy link
Copy Markdown
Member Author

This needs more coverage, but manually testing looks good

@bdraco bdraco marked this pull request as draft October 22, 2021 21:56
@bdraco

bdraco commented Jan 26, 2022

Copy link
Copy Markdown
Member Author

We can do this part. But part 2 probably isn't going to happen unless I can figure out how to avoid a breaking change

1 similar comment
@bdraco

bdraco commented Jan 26, 2022

Copy link
Copy Markdown
Member Author

We can do this part. But part 2 probably isn't going to happen unless I can figure out how to avoid a breaking change

@bdraco bdraco force-pushed the ignore_stale_records_asking branch from 9b96f9c to 7f2de64 Compare January 31, 2022 23:14
@bdraco

bdraco commented Dec 16, 2023

Copy link
Copy Markdown
Member Author

fixed via #1348

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants