search

pangeo-forge / pangeo-forge-orchestrator

Database API and GitHub App backend for Pangeo Forge Cloud.
https://api.pangeo-forge.org/docs
Apache License 2.0
4 stars 1 forks source link

Don't `deploy_prod_run` if PR doesn't change `feedstock/` #92

Closed cisaacstern closed 2 years ago

cisaacstern commented 2 years ago

A variant of #91, but for feedstock repos (rather than staged-recipes).

I've just been starting to delete the GitHub Workflows on our existing feedstock repos (because we no longer need them, with the new GitHub App), e.g., https://github.com/pangeo-forge/paleo-pism-feedstock/pull/4.

Without first completing this todo item, however, I've realized these PRs will deploy production runs for the feedstocks:

https://github.com/pangeo-forge/pangeo-forge-orchestrator/blob/5b46a93fbe1e1064fbefd8156e8207b3d5b43323/pangeo_forge_orchestrator/routers/github_app.py#L424-L426

I'll work on a PR to resolve this todo before proceeding with deleting the workflows.

cisaacstern commented 2 years ago

Fixed by #100