feat: PR triggers - Githubissues

bcgov / nr-spar

SPAR app from FDS (Forestry Digital Services)

Apache License 2.0

1 stars 0 forks source link

Closed DerekRoberts closed 2 days ago

DerekRoberts commented 1 month ago

At some point during our revisions PR deploy triggers were dropped. Restore them, so non-code changes won't trigger builds and deploys.

New request: only deploy fluentbit when log_env_target is provided.

RMCampos commented 1 month ago

Ideally, we could have something like this, for each component:

PR Open - DEV

PR Closed - Success Merge

If Changes ➡️ then, triggers both build and deployments
If no changes ➡️ then, skip everything on this component, both build and deploy

DerekRoberts commented 1 month ago

@RMCampos That's workable! Are we treating Oracle-API any differently? I think everything else makes sense to build and deploy as a group.

RMCampos commented 1 month ago

@DerekRoberts I'd say to treat Oracle-API the same way.