Remove branch build configuration from appveyor.yml

cmu-cs-academy / desktop-cmu-graphics

BSD 3-Clause "New" or "Revised" License

16 stars 10 forks source link

Remove branch build configuration from appveyor.yml #18

Closed schmave closed 3 years ago

schmave commented 3 years ago

Seems like it should be fine to build all branches, and in fact that's good to verify that PRs work properly. Any reason not to do this?

austin-schick commented 3 years ago

How are deploy builds working now? Is it possible for a PR build to set up an artifact that gets accidentally deployed by CodeBuild?

schmave commented 3 years ago

PR builds from the add-docs branch no longer generate artifacts. The CodeBuild deploy job doesn't use artifacts to deploy and isn't automatically triggered by anything.