-
Notifications
You must be signed in to change notification settings - Fork 104
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
Move code tests to code dirs (attempt 2) #1022
base: main
Are you sure you want to change the base?
Conversation
If this PR is merged, the tests for a code will be available under |
Assuming the tests will all be in Python, then making a |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 28 days if no further activity occurs. Thank you for your contributions. |
Hmm, I think this is a good idea, and it's tempting to do it while working on the new build system. I will want to test things anyway to reduce the chances of breaking anything, and since I'm setting up a new system to build and install, I might as well add running tests to it. If it's just moving the files... |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 28 days if no further activity occurs. Thank you for your contributions. |
This is now being done as part of the move to the new build system. There seem to be a bunch of unrelated updates in the final commit related to the build system and the CI, and whatever that does will have to be done there as well anyway. So I propose we close this? |
The idea is that the tests for a specific community code (which is available as a separate package) should be included with that code, and not in a centralised place.