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

Update some warnings with instructions and version number. #761

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

Carreau
Copy link
Member

@Carreau Carreau commented Sep 1, 2021

I came across this when trying to reproduce an unrelated bug with
someone in an outdated environment, but at least having the version
number will make us more confident when e remove the conditional branch
later.

I came across this when trying to reproduce an unrelated bug with
someone in an outdated environment, but at least having the version
number will make us more confident when e remove the conditional branch
later.
@Carreau Carreau requested a review from minrk September 1, 2021 18:07
@Carreau
Copy link
Member Author

Carreau commented Sep 1, 2021

@minrk There seem to be unexpected crash with a NameError should_crash when testing downstream ipyparallel 7.0.0b2, not sure if you want to have a look.

@minrk minrk merged commit 3c6037f into ipython:master Sep 1, 2021
@minrk
Copy link
Member

minrk commented Sep 1, 2021

Weird, I'll look into it. Looking at the test, that would normally be caused by cloudpickle somehow being invoked.

@blink1073 blink1073 added this to the 6.4 milestone Sep 9, 2021
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

3 participants