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

Cleanup FPS Project for 0.3.0 #239

Merged
merged 14 commits into from
Nov 11, 2019
Merged

Cleanup FPS Project for 0.3.0 #239

merged 14 commits into from
Nov 11, 2019

Conversation

zeroZshadow
Copy link
Contributor

@zeroZshadow zeroZshadow commented Nov 8, 2019

Description

  • Moved all "package" code and schema into the project directly.
    • Flattened the Fps/Scripts structure
  • Cleaned up the namespaces to be clearer that this is game specific schema and code.
  • Simplified some over-generalised code.

Would highly recommend reviewers should checkout this branch and browse through the structure as part of the review process.

Documentation

  • Changelog

@improbable-prow-robot improbable-prow-robot added jira/no-ticket Indicates a PR has no corresponding JIRA ticket do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 600+ lines, ignoring generated files. labels Nov 8, 2019
@zeroZshadow zeroZshadow marked this pull request as ready for review November 8, 2019 13:58
@improbable-prow-robot improbable-prow-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 8, 2019
Paul Balaji added 4 commits November 8, 2019 16:02
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
Copy link
Contributor

@jessicafalk jessicafalk left a comment

Choose a reason for hiding this comment

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

took a look at it, moving the files looks good and everything still runs as expected ;)
in the future we should look into whether we can get rid of the movement stuff we did here and replace it with the transform sync and also maybe move all prefabs into one folder (right now we still them in two places depending on how they are used)

Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
CHANGELOG.md Outdated Show resolved Hide resolved
Co-Authored-By: Martijn Gerkes <martijn@improbable.io>
Copy link
Contributor

@paulbalaji paulbalaji left a comment

Choose a reason for hiding this comment

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

Approving on Martijn's behalf

image

@paulbalaji paulbalaji merged commit 7a58843 into develop Nov 11, 2019
@paulbalaji paulbalaji deleted the feature/scorched-earth branch November 11, 2019 12:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
jira/no-ticket Indicates a PR has no corresponding JIRA ticket size/XXL Denotes a PR that changes 600+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants