Skip to content

Conversation

ieva-improbable
Copy link
Contributor

Description

Basic setup of Example Project CI. Example project gets built using the latest of GDK master or the GDK branch specified when kicking off the build. It builds Editor, generates schema and snapshot, builds Linux worker and Windows Client. If the build is successful, the built out workers get uploaded and a new deployment is launched in unreal_gdk project.

Note: this PR requires spatialos/UnrealGDK#1360 to be merged in first.

…d once again, setting the default in steps was overwriting the custom values.
@improbable-prow-robot improbable-prow-robot added the size/L Denotes a PR that changes 150-299 lines, ignoring generated files. label Sep 24, 2019
@improbable-prow-robot improbable-prow-robot added size/XL Denotes a PR that changes 300-599 lines, ignoring generated files. and removed size/L Denotes a PR that changes 150-299 lines, ignoring generated files. labels Sep 24, 2019
Copy link

@padhansell padhansell left a comment

Choose a reason for hiding this comment

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

LGTM

ieva-improbable and others added 8 commits September 25, 2019 11:48
Co-Authored-By: Patrick Hansell <patrickhansell@hotmail.co.uk>
Co-Authored-By: Patrick Hansell <patrickhansell@hotmail.co.uk>
Co-Authored-By: Patrick Hansell <patrickhansell@hotmail.co.uk>
"-MapPaths=`"/Maps/FPS-Start_Small`""
)

$core_gdk_schema_path = "$($gdk_home)\SpatialGDK\Extras\schema\*"
Copy link
Collaborator

Choose a reason for hiding this comment

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

These schema copying steps are done within setup.bat/setup.sh. Is there a reason we aren't using that script here?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a comment to fix this in future.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ticket number: GV-29

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, thanks @mironec ! I'll link that.

@ieva-improbable ieva-improbable merged commit 4d5dc72 into master Sep 26, 2019
@m-samiec m-samiec deleted the feature/ci branch September 29, 2019 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jira/UNR size/XL Denotes a PR that changes 300-599 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants