Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add trio xfail #82

Merged
merged 1 commit into from
Jun 27, 2023
Merged

Add trio xfail #82

merged 1 commit into from
Jun 27, 2023

Conversation

jacobtomlinson
Copy link
Member

Currently we do not support trio, but would like to get there at some point. This PR adds trio to the test suite and an xfailed test that uses trio.

#77 will track us working towards getting this test passing.

@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Merging #82 (3fcef05) into main (11db1e0) will decrease coverage by 0.09%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main      #82      +/-   ##
==========================================
- Coverage   96.85%   96.76%   -0.09%     
==========================================
  Files          22       23       +1     
  Lines        1878     1888      +10     
==========================================
+ Hits         1819     1827       +8     
- Misses         59       61       +2     
Impacted Files Coverage Δ
kr8s/tests/test_anyio.py 80.00% <80.00%> (ø)

@jacobtomlinson jacobtomlinson merged commit 4e4c233 into kr8s-org:main Jun 27, 2023
11 of 12 checks passed
@jacobtomlinson jacobtomlinson deleted the trio-xfail branch June 27, 2023 10:54
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.

None yet

1 participant