Page MenuHomeElementl

limit azure runs to main branch
ClosedPublic

Authored by alangenfeld on Jul 22 2021, 7:31 PM.

Details

Summary

Building on all branches doesn't make sense, especially once we start pusing more branches.

PRs will still run it - though on our current plan we may find this unsustainable as well.

Test Plan

eyes

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cancel pr builds when pr updated

dgibson added a subscriber: dgibson.

yessssss does this mean i can turn on alerts in azure-pipelines?! amazing

This revision is now accepted and ready to land.Jul 22 2021, 8:01 PM

PRs might make that noisy-ish still but we may have to disable those as well

This revision was automatically updated to reflect the committed changes.

welp - this wasnt quite right. Followed up with
https://github.com/dagster-io/dagster/commit/66a185240a7b7137d8fb97beaabd3b3d93ae9b4e
where i just disabled PRs as well for now