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

Conversation

@paulbalaji
Copy link
Contributor

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.


Description

adds navmesh to FPS-SimulatedPlayerCoordinator scene

Tests

matches the one in the FPS-Development scene

Documentation

none needed

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.

Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
@paulbalaji paulbalaji changed the title add navmesh to SimulatedPlayerCoordinator scene Add navmesh to SimulatedPlayerCoordinator scene Oct 10, 2018
@paulbalaji paulbalaji changed the title Add navmesh to SimulatedPlayerCoordinator scene Bake navmesh to SimulatedPlayerCoordinator scene Oct 10, 2018
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
Copy link
Contributor

@jamiebrynes7 jamiebrynes7 left a comment

Choose a reason for hiding this comment

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

I trust you

@paulbalaji paulbalaji merged commit 600a3d0 into master Oct 10, 2018
@paulbalaji paulbalaji deleted the fix/add-navmesh-to-simulated-player-scene branch October 10, 2018 13:53
BenNaccarato pushed a commit that referenced this pull request Dec 14, 2018
**Contributions**: We are not currently taking public contributions - see our [contributions](https://github.com/spatialos/UnityGDK/blob/master/CONTRIBUTING.md) policy. However, we are accepting issues and we do want your [feedback](https://github.com/spatialos/UnityGDK/blob/master/README.md#give-us-feedback).

-------

#### Description
Couple small changes:
* Enforce local links for links in the repository. This makes the linking more reliable and easier to test (for example a remote link to a invalid heading in a markdown file will not fail, whereas local links will) as well as running through the linter faster.
* Allowed local links to directories
* Added more logging for when a remote link check throws an exception.
* Made changes to all the `md` files that were in violation of the first point. 
* Added dotenv file to track the remote url. This is used for the enforcement of links and in the docs verification branch.

#### Tests
Ran `ci/docs-lint.sh` and added tests to the project for the new conditions.
#### Documentation
This is documentation? We might want to add a style guide for markdown to enforce local links.
#### 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.
@firtina-improbable @ElleEss
BenNaccarato added a commit that referenced this pull request Dec 14, 2018
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