Skip to content

Add per-PR CI running pytest#12

Merged
garthwatney merged 1 commit intomainfrom
rosemurg-ci
Apr 27, 2026
Merged

Add per-PR CI running pytest#12
garthwatney merged 1 commit intomainfrom
rosemurg-ci

Conversation

@pcrosemurgy
Copy link
Copy Markdown
Collaborator

@pcrosemurgy pcrosemurgy commented Apr 27, 2026

This PR adds a CI check that runs pytest for every PR.

See https://github.com/JPLOpenSource/STARS/pull/12/checks for CI output for this PR.

@pcrosemurgy pcrosemurgy force-pushed the rosemurg-ci branch 4 times, most recently from 9765e45 to 6b344d8 Compare April 27, 2026 18:47
@pcrosemurgy pcrosemurgy requested a review from garthwatney April 27, 2026 18:49
@pcrosemurgy pcrosemurgy marked this pull request as ready for review April 27, 2026 18:50
Comment thread requirements.txt
pyparsing==3.1.1
plantuml==0.3.0
pydantic==2.6.3
pyyaml==6.0.3
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When running the GitHub Action above I got "module not found: yaml" errors.

Looking at requirements.txt I believe this is a valid error as there was no explicit pyyaml package defined. Likely this has not been an issue since all dev hosts STARS has been tested on has had pyyaml packages cached in the user's pip cache.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has been fixed in 476b4dc

@garthwatney garthwatney merged commit 26d206a into main Apr 27, 2026
1 check passed
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.

2 participants