Skip to content

Try without dep groups#356

Merged
Cadair merged 2 commits intoOpenAstronomy:mainfrom
Cadair:coverage-dep-groups
Feb 26, 2026
Merged

Try without dep groups#356
Cadair merged 2 commits intoOpenAstronomy:mainfrom
Cadair:coverage-dep-groups

Conversation

@Cadair
Copy link
Member

@Cadair Cadair commented Feb 25, 2026

Experimenting with not using dep groups in the tox workflow itself, see #207 (comment)

@neutrinoceros @zacharyburnett thoughts?

@neutrinoceros
Copy link
Contributor

Makes sense. But then why install it at all when you could run it through pipx ?

@neutrinoceros
Copy link
Contributor

neutrinoceros commented Feb 25, 2026

(pipx itself is included in actions/setup-python)

@Cadair Cadair merged commit 9ed6008 into OpenAstronomy:main Feb 26, 2026
70 checks passed
@Cadair Cadair deleted the coverage-dep-groups branch February 26, 2026 09:38
@zacharyburnett
Copy link
Contributor

this makes this much cleaner!

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.

3 participants