Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Appveyor hooks for Unity Cloudbuild #6

Merged
merged 16 commits into from
Apr 3, 2017
Merged

Appveyor hooks for Unity Cloudbuild #6

merged 16 commits into from
Apr 3, 2017

Conversation

isiyu
Copy link
Contributor

@isiyu isiyu commented Apr 1, 2017

  • Appveyor builds and pushes a clean sdkpackage (with submodules) to private repo
  • Appveyor decodes and writes token under Assets/StreamingAssets

TODO:

  • attach hooks to trigger Unity Cloud Build

wilhelmberg and others added 16 commits March 31, 2017 11:13
@wilhelmberg
test [trigger cloudbuild]
a33d3c2
@wilhelmberg
test2 [trigger cloudbuild]
cbc0ca7
@wilhelmberg
test3 [trigger cloudbuild]
14dac3a
@wilhelmberg
test4 [trigger cloudbuild], AppVeyor: exclude 'unity-cloud-build' to …
71baab6 
…prevent recursive triggering of AppVeyor builds
@wilhelmberg
test5 [trigger cloudbuild], Appveyor does not honor branches:except
5548370
@wilhelmberg
test6 [trigger cloudbuild], Appveyor does not honor branches:except
b44f10a
@wilhelmberg
test8 [trigger cloudbuild], delete .gitignore in 'sdkproject'
c73455e
@isiyu
test9 [trigger cloudbuild], change cloud build to private repo.
0134f32
@isiyu
test9 [trigger cloudbuild], writing out token file for mapboxaccess t…
55c536b 
…oken for cloudbuild
@isiyu
double quotes
6d9fdb5
@isiyu
test10 [trigger cloudbuild], test double quotes fix
50f5569
@isiyu
test11 [trigger cloudbuild], test syntax fix
094d355
@isiyu
test12 [trigger cloudbuild], test syntax fix
4ad77b6
@isiyu
test13 [trigger cloudbuild], test syntax fix
e832927
@isiyu
test14 [trigger cloudbuild], test syntax fix
5849bec
@isiyu
removing print statement
31028dc
@isiyu isiyu mentioned this pull request Apr 1, 2017
Closed
4 tasks
david-rhodes
david-rhodes approved these changes Apr 3, 2017
View reviewed changes
Copy link

@david-rhodes david-rhodes left a comment

Choose a reason for hiding this comment

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

Working fine. Cloud build side needs love, but this is good for develop.

@david-rhodes david-rhodes merged commit 91eb8ba into develop Apr 3, 2017
@david-rhodes david-rhodes deleted the BergWerkGIS-prepare-for-cloudbuild branch April 3, 2017 17:51
wilhelmberg
wilhelmberg reviewed Apr 4, 2017
View reviewed changes
scripts/build.csx Outdated
@@ -164,8 +164,6 @@ if (!triggerCloudBuild) {
"git push -f origin CloudBuild"
});
foreach (var cmd in cmds) {
Console.WriteLine("RUNNING COMMAND:");
Console.WriteLine(cmd);
Copy link
Contributor

Choose a reason for hiding this comment

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

@isiyu for reference, if this is ever needed again:
You can pass showCmd:true to RunCommand()
😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wilhelmberg wilhelmberg wilhelmberg left review comments

@david-rhodes david-rhodes david-rhodes approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@isiyu @wilhelmberg @david-rhodes