Skip to content

Skip some PyPy tests and revert breaking poll_at timestamp change#621

Merged
wbarnha merged 7 commits into
masterfrom
fix-pypy-tests
Apr 2, 2024
Merged

Skip some PyPy tests and revert breaking poll_at timestamp change#621
wbarnha merged 7 commits into
masterfrom
fix-pypy-tests

Conversation

@wbarnha

@wbarnha wbarnha commented Apr 2, 2024

Copy link
Copy Markdown
Member

Note: Before submitting this pull request, please review our contributing
guidelines
.

Description

Made a mistake in #610 where the poll_at timestamp should have been checked since it's in milliseconds according to https://github.com/aure-olli/aiokafka/blob/master/aiokafka/consumer/fetcher.py#L677.

@codecov

codecov Bot commented Apr 2, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.08%. Comparing base (2c4c8bb) to head (b45964d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #621   +/-   ##
=======================================
  Coverage   94.08%   94.08%           
=======================================
  Files         102      102           
  Lines       11117    11117           
  Branches     1550     1550           
=======================================
  Hits        10459    10459           
  Misses        558      558           
  Partials      100      100           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@wbarnha wbarnha marked this pull request as ready for review April 2, 2024 19:26
@wbarnha wbarnha merged commit a489db3 into master Apr 2, 2024
@wbarnha wbarnha deleted the fix-pypy-tests branch April 2, 2024 19:31
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