Mock import of tests module from package

MDAnalysis / cookiecutter-mdakit

Cookiecutter for Python packages based on MDAnalysis

MIT License

7 stars 5 forks source link

Mock import of tests module from package #89

Closed ianmkenney closed 1 year ago

ianmkenney commented 1 year ago

Fixes #81

PR Checklist

[ ] Tests?
[ ] Docs?
[ ] CHANGELOG.md updated?
[ ] AUTHORS.md updated?
[ ] Issue raised/referenced?

lilyminium commented 1 year ago

Thanks @ianmkenney. Could you please add a line commenting what this does in the file?

If you've tested this locally by making a cookie and creating docs, would you mind uploading that to this PR as well? It's not somehting we look for in the CI (although perhaps it should be).

ianmkenney commented 1 year ago

@lilyminium I ran into it when making the mdaencore docs. I couldn't find any other way to ignore a module in the autosummary directive.

lilyminium commented 1 year ago

Thanks for merging, @ianmkenney! Just for next time, a "Request changes" review is conventionally blocking and requires resolution and re-review before merging -- my fault for not setting up the ruleset on this repo before this PR, but just explaining why I've done so now.