Skip to content

Conversation

@sbillinge
Copy link
Contributor

@cadenmyers13 could you make a new PR and try and make a solution that looks more like this to work?

The basic idea is to copy the entire docs/examples directory to the pytest tmdir fixture, then run everything from there.
Please double check that all output files are written to the tmpdir At least in the NI example they are because they write to PWD/... and PWD is Path('__file__').parent, so as long as we discover the *.py file in the tmpdir copy we should be good to go. Nice aspect is that the code for this is super-simple.

Maybe lets add psutils to requirements/tests.txt rather than comment it out in the example. You will have to do the headless thing etc., but I think this could work, something like this.

@github-actions
Copy link

Warning! No news item is found for this PR. If this is a user-facing
change/feature/fix,
please add a news item by copying the format from news/TEMPLATE.rst.
For best practices, please visit
https://scikit-package.github.io/scikit-package/frequently-asked-questions.html#billinge-group-standards.

@cadenmyers13
Copy link
Contributor

@sbillinge yeah I'll give it a shot. Thanks for the template!

@sbillinge sbillinge merged commit c58bb59 into diffpy:main Sep 23, 2025
1 of 3 checks passed
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