decaporg / decap-cms

A Git-based CMS for Static Site Generators
https://decapcms.org
MIT License
17.93k stars 3.04k forks source link

Move if-s to job-level in CI #7033

Closed szepeviktor closed 8 months ago

szepeviktor commented 10 months ago

Summary

Move if-s to job-level in CI reducing the number of if-s to two.

Checklist

Please add a x inside each checkbox:

A picture of a cute animal (not mandatory but encouraged)

image

netlify[bot] commented 10 months ago

Deploy Preview for decap-www ready!

Name Link
Latest commit e72632f075168ab5003794e39fe50328f9cc4a81
Latest deploy log https://app.netlify.com/sites/decap-www/deploys/65a67324be418900080b7239
Deploy Preview https://deploy-preview-7033--decap-www.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

martinjagodic commented 9 months ago

@szepeviktor can you explain the motivation behind this PR?

szepeviktor commented 9 months ago

Yes. No need for conditions in each step. Just add one condition to each job.

martinjagodic commented 8 months ago

@szepeviktor do you mind fixing the conflict? Sorry it took so long.

szepeviktor commented 8 months ago

@martinjagodic Done. No problem.

szepeviktor commented 8 months ago

🍓