Update copier and dependencies #485
Conversation
| @@ -0,0 +1,83 @@ | |||
| from typing import Any, NewType, Optional | |||
There was a problem hiding this comment.
Can you explain what the code in this file is doing? How differently do things render in the docs when this is used?
There was a problem hiding this comment.
| - tox==4.11.3 | ||
|
|
||
| # docs |
There was a problem hiding this comment.
Why were the docs deps removed? Is it because we no longer want to build the docs on py39?
There was a problem hiding this comment.
We can't build on 3.9 because there is no compatible mantid version. So I simplified the file
| scippneutron | ||
| pytest | ||
| commands = pytest | ||
|
|
||
| [testenv:static] |
There was a problem hiding this comment.
Not related to this line
Are we running too many checks? There seems to be a mix of ubuntu versions, and python versions? Do we still have old CI workflows running alongside new ones?
There was a problem hiding this comment.
True. We have both ci.yml and pr_and_main.yml. Must have been a messed up merge at some point.
There was a problem hiding this comment.
I thought this was intentional, to have builds with Mantid (using conda) as well as the common setup (no Mantid).
Co-authored-by: Neil Vaytet <[email protected]>
No description provided.