Move tests directory to root level#129
Move tests directory to root level#129Kludex merged 6 commits intopy-bson:masterfrom stankudrow:make-tests-dir
Conversation
|
Can we start the changes on the test with the GitHub actions running the test suite? So we are sure we don't break anything here... |
|
Yes, we should -- I started working on that, but there's a bit of chicken-egg thing here -- I think we want to update the build system first, so we can use that in the gitHub actions ... Does anyone have a PR for updating the build already? (e.g. going to hatchling and uv). |
|
Ahh I see #126 -- but that's using PDM -- we want to go hatchling and UV, yes? |
In short, we can. Just my remark: the changes of this PR are mostly cosmetic and we don't need to wait for GHA workflows because, for instance, I cannot see any breaking changes. But if ideologically the GHA is the first thing, this PR can wait. |
Do you wanna go with the |
ChrisBarker-NOAA
left a comment
There was a problem hiding this comment.
looks like we have a dependency problem:
ModuleNotFoundError: No module named 'dateutil'
Probably need to add an install step before running tests -- that should install the dependencies.
|
nice! |
Partially covers: #126 - rearranging the tests
Partially addresses: #127