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

fix: ignore platform/version specific trio failures #3922

Closed
wants to merge 1 commit into from

Conversation

cofin
Copy link
Member

@cofin cofin commented Jan 4, 2025

Description

This PR temporarily ignores failures related to trio and specific anyio versions on Windows platforms.

Here is an example of the related error

Closes

Copy link

codecov bot commented Jan 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.36%. Comparing base (75d3430) to head (4202e65).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3922   +/-   ##
=======================================
  Coverage   98.36%   98.36%           
=======================================
  Files         347      347           
  Lines       15722    15722           
  Branches     1737     1737           
=======================================
  Hits        15465    15465           
  Misses        122      122           
  Partials      135      135           

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

@cofin cofin enabled auto-merge (squash) January 5, 2025 01:30
Copy link

sonarqubecloud bot commented Jan 5, 2025

Copy link

github-actions bot commented Jan 5, 2025

Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/3922

@cofin
Copy link
Member Author

cofin commented Jan 5, 2025

I think this is no longer needed after the pytest upgrade here #3925

@cofin cofin closed this Jan 5, 2025
auto-merge was automatically disabled January 5, 2025 03:17

Pull request was closed

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.

1 participant