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

More Informative assert #1314

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

More Informative assert #1314

wants to merge 1 commit into from

Conversation

Carreau
Copy link
Member

@Carreau Carreau commented Jan 21, 2025

This keep failing downstream in IPython. It's annoying. I hope this will give an idea as to why. Otherwise can we mark the test as xfail ?

If not I"ll stop testing in IPython if I break ipykernel.

This  keep failing downstream in IPython. It's annoying.
I hope this will give an idea as to why. Otherwise can we mark the test as xfail ?

If not I"ll stop testing in IPython if I break ipykernel.
@Carreau Carreau requested a review from ianthomas23 January 21, 2025 15:39
Copy link
Collaborator

@ianthomas23 ianthomas23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am happy to make this change permanently, or the xfail alternative if you'd prefer on a temporary basis.

There seem to be quite a lot of upstream and downstream failures in various related projects CI. This test here is somewhat risky as it is timing based and is probably best to relax it a bit, but I don't want to spend any more time on the subshells work here until the anyio changes are completed.

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