Skip to content

Fix pytest8 jupyternbcollector#83

Open
ibressler wants to merge 4 commits intochrisjsewell:masterfrom
BAMresearch:fix-pytest8-JupyterNbCollector
Open

Fix pytest8 jupyternbcollector#83
ibressler wants to merge 4 commits intochrisjsewell:masterfrom
BAMresearch:fix-pytest8-JupyterNbCollector

Conversation

@ibressler
Copy link
Copy Markdown

No description provided.

@ibressler ibressler mentioned this pull request May 7, 2024
@ibressler
Copy link
Copy Markdown
Author

The relevant part here is commit b70c440, the previous commits are needed to make the PR checks pass, related to #82

@tlunet
Copy link
Copy Markdown

tlunet commented Nov 14, 2025

Is this going to be merged ? Tests are now failing with pytest=9.0.0, and those small changes should probably solve it ...

@amotl
Copy link
Copy Markdown
Contributor

amotl commented Dec 15, 2025

Hi. We also need this patch for supporting compatibility with pytest 9. 🙏

@ibresslerBAM
Copy link
Copy Markdown

yes, because of the inactivity here I am using my own fork in testing active projects:
https://github.com/BAMresearch/pytest-notebook
And in requirements.txt I reference it like that: https://github.com/BAMresearch/yapy-copier-template/blob/main/%7B%7Brepo_name%7D%7D/tests/requirements.txt (from my copier project template)

Don't know, could publish this for as pypi package too, but not under the original name - don't know a good alternative name :/

@amotl
Copy link
Copy Markdown
Contributor

amotl commented Dec 24, 2025

Thank you. Maybe we can use your fork in the meanwhile. If maintenance can not be resumed on the original project, maybe indeed publish pytest-notebook2. But first, let's ask @chrisjsewell about his plans. 🍀

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.

4 participants