Skip to content

Dependencies: relax requirement nest-asyncio~=1.5#229

Merged
sphuber merged 1 commit intodevelopfrom
fix/update-nest-asyncio
Apr 8, 2022
Merged

Dependencies: relax requirement nest-asyncio~=1.5#229
sphuber merged 1 commit intodevelopfrom
fix/update-nest-asyncio

Conversation

@sphuber
Copy link
Copy Markdown
Collaborator

@sphuber sphuber commented Apr 8, 2022

It was pinned to a patch version which was causing conflicts with other
dependencies for aiida-core, notably notebook.

It was pinned to a patch version which was causing conflicts with other
dependencies for `aiida-core`, notably `notebook`. We update instead of
relaxing to the `1.4` minor version, because `v1.5` contains
incompatible changes in the reset of the event loop policy.
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 8, 2022

Codecov Report

Merging #229 (ce0ba63) into develop (5685c6c) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #229      +/-   ##
===========================================
- Coverage    90.58%   90.56%   -0.01%     
===========================================
  Files           22       22              
  Lines         2980     2974       -6     
===========================================
- Hits          2699     2693       -6     
  Misses         281      281              
Impacted Files Coverage Δ
plumpy/events.py 88.71% <ø> (-0.99%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5685c6c...ce0ba63. Read the comment docs.

@sphuber sphuber force-pushed the fix/update-nest-asyncio branch from 992eb26 to ce0ba63 Compare April 8, 2022 12:18
@sphuber sphuber requested review from chrisjsewell and csadorf April 8, 2022 12:41
@sphuber
Copy link
Copy Markdown
Collaborator Author

sphuber commented Apr 8, 2022

@csadorf only noticed #223 and your comment when I had already fixed it.

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Apr 8, 2022

@csadorf only noticed #223 and your comment when I had already fixed it.

ok, yeah, thanks. this has been sitting in my todo list for too long

@sphuber sphuber merged commit c500b99 into develop Apr 8, 2022
@sphuber sphuber deleted the fix/update-nest-asyncio branch April 8, 2022 12:57
@sphuber sphuber mentioned this pull request Apr 8, 2022
@sphuber sphuber changed the title Dependencies: relax requirement nest-asyncio~=1.4 Dependencies: relax requirement nest-asyncio~=1.5 Apr 8, 2022
unkcpz pushed a commit to unkcpz/plumpy that referenced this pull request Dec 14, 2024
It was pinned to a patch version which was causing conflicts with other
dependencies for `aiida-core`, notably `notebook`. We update instead of
relaxing to the `1.4` minor version, because `v1.5` contains
incompatible changes in the reset of the event loop policy.
agoscinski pushed a commit to agoscinski/plumpy that referenced this pull request Apr 13, 2026
It was pinned to a patch version which was causing conflicts with other
dependencies for `aiida-core`, notably `notebook`. We update instead of
relaxing to the `1.4` minor version, because `v1.5` contains
incompatible changes in the reset of the event loop policy.
agoscinski pushed a commit that referenced this pull request Apr 13, 2026
It was pinned to a patch version which was causing conflicts with other
dependencies for `aiida-core`, notably `notebook`. We update instead of
relaxing to the `1.4` minor version, because `v1.5` contains
incompatible changes in the reset of the event loop policy.
agoscinski pushed a commit that referenced this pull request Apr 13, 2026
It was pinned to a patch version which was causing conflicts with other
dependencies for `aiida-core`, notably `notebook`. We update instead of
relaxing to the `1.4` minor version, because `v1.5` contains
incompatible changes in the reset of the event loop policy.
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.

2 participants