scullyio / scully

The Static Site Generator for Angular apps
https://scully.io/
MIT License
2.55k stars 257 forks source link

devops, make workflow do publish and test only on PR #1255

Open SanderElias opened 3 years ago

SanderElias commented 3 years ago

Discuss

pjlamb12 commented 3 years ago

Is this the workflow.yaml file, and just editing it to only run on PR to main instead of push?

SanderElias commented 3 years ago

yes it is the workflow.yaml. but right now, its running on every push. and doing all f the work. It should be changed so, that it only runs the last 2 steps on PR. We don't want to publish new @develop version on each push, but we do on every pr. The same goes for the Can we build test step, that is only needed on pr.