Skip to content
This repository was archived by the owner on Oct 20, 2021. It is now read-only.

Conversation

paulbalaji
Copy link
Contributor

very very similar to the changes here: spatialos/gdk-for-unity-fps-starter-project#115

Description

  • added premerge pipeline
  • added release-qa pipeline
  • use the build.sh that's been moved to gdk-for-unity-shared-ci
  • rename ci/upload-launch.sh to ci/launch.sh

Tests

ran through buildkite and TC, looks green to me

Documentation

n/a

Primary reviewers

If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

Paul Balaji added 2 commits February 26, 2019 18:37

uploadAssembly "${ASSEMBLY_PREFIX}" "${PROJECT_NAME}"

markStartOfBlock "Launching deployments"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given this is only running on BK we can drop these (as they are TC specific markers)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will need to go through all our ci scripts properly when we fully migrate off TC 👍

@@ -1,22 +0,0 @@
#!/usr/bin/env bash
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably want to remove the Release QA pipeline on TC since we've removed this. Or restore it and make a note to remove it when we kill TC

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing Release QA pipeline from TC as it looks like it's not actually being run

@paulbalaji paulbalaji merged commit e2324e3 into develop Feb 28, 2019
@paulbalaji paulbalaji deleted the feature/buildkite-pipelines branch February 28, 2019 17:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants