fix pip install in CI

[zacharyburnett]

use pip install .[test] @mfixstsci

[mfixstsci]

@zacharyburnett would it be possible, or make sense, to remove the line @pllim and I are talking about in #279? It would be nice to see if the tests are still passing once that is removed. Thanks!

[pllim]

LGTM. Yes, Python 3.7 has reached EOL and should be removed, which you did.

[pllim]

And yes, would be nice to remove oldest-supported-numpy though it is technically out of scope here. Depends on how many PRs you feel like opening today. Has to be done, but can be here or a follow-up.

[zacharyburnett]

sure, I'll remove oldest-supported-numpy (not sure why I added it in the first place, sorry), and see

[zacharyburnett]

Ok, this looks ready to me

[mfixstsci]

Boom, still passing. Thank you @zacharyburnett

[mfixstsci]

LGTM! Need to update docs to make sure we are using python >=3.9

[pllim]

A grep like this in root dir should smoke most of them out.

grep -ri python . | grep "3\.7"

[zacharyburnett]

Numpy also dropped supporting Python 3.8 in new versions; should the required Python version in pyproject.toml also be updated?

pysiaf/pyproject.toml

Line 4 in f49b529

requires-python = ">=3.8"

There is also a reference in README.md:

pysiaf/README.md

Line 136 in f49b529

This package is supported in python 3.8 and 3.9

[pllim]

Numpy also dropped supporting Python 3.8 in new versions

Then Python 3.8 would pick up an older numpy. Up to you whether you want to drop it or not here, but you do not have to. Though I think astropy also dropped it for upcoming v6.