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

add aiida-testing #206

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

ltalirz
Copy link
Member

@ltalirz ltalirz commented Jan 28, 2022

No description provided.

@ltalirz
Copy link
Member Author

ltalirz commented Jan 28, 2022

@chrisjsewell I'd like to add this plugin here since I've been using it and it can be quite handy (e.g. when testing codes that have long runtimes / are difficult to install on CI).

As the docs describe in more detail there are two aspects to it - it can mock codes + it can speed up tests via caching.

Nevertheless, the name aiida-testing is very generic, so I'd just like a second opinion on whether that is considered acceptable or whether we need to go through the process of changing it to something else.

@chrisjsewell
Copy link
Member

Errm, I guess I would say aiida-testing is more of a "tool" than a plugin, although not necessarily against it being on here; I would think its more something to advertise/suggest in the docs (on how to write plugins)

@ltalirz
Copy link
Member Author

ltalirz commented Jan 28, 2022

It's true that it currently does not provide any entry points.
On the other hand, I think it would make sense to advertise it here.

I'll bring it up some point in the AiiDA meeting and will hold the PR off until then.

@ltalirz ltalirz marked this pull request as draft January 28, 2022 14:50
@chrisjsewell
Copy link
Member

Yeh no worries, indeed its good stuff though, so should definitely be advertised

@unkcpz unkcpz force-pushed the master branch 6 times, most recently from 1b42871 to d2889c8 Compare December 19, 2023 02:46
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