Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Conversation

@blake-lowe
Copy link
Contributor

@blake-lowe blake-lowe commented Aug 23, 2018

Description of the Change

Updated documentation in getting-started.md
Removed unfinished 4th step for # setting up a new repository
Deleted # connecting to an existing repository
I'm not sure when this case would arise as the proper workflow should have the unity project with plugin in a repository, never the project alone.
Added steps for # Connecting to an Existing Repository that already has the GitHub for Unity package
These are based on what worked for me today.

Alternate Designs

You may want to not remove the header # Connecting to an Existing Repository for some reason

Benefits

Clarity for new users who wish to use multiple machines to work on a project (the main functionality of GitHub)

Possible Drawbacks

None, just editing text here

Applicable Issues

Issue #891

Removed unfinished 4th step for # setting up a new repository
Deleted # connecting to an existing repository
    I'm not sure when this case would arise as the proper workflow should have the unity project with plugin in a repository, never the project alone.
Added steps for # Connecting to an Existing Repository that already has the GitHub for Unity package
     These are based on what worked for me today.
@blake-lowe
Copy link
Contributor Author

Did just notice that the top of the file has a link to that header. So up to you to delete or save it

@StanleyGoldman
Copy link
Contributor

Hey @dragonfyre23 I'm going to merge your commit to #897 and make some more tweaks.

@StanleyGoldman StanleyGoldman changed the base branch from master to documentation-fixes August 27, 2018 14:57
@StanleyGoldman StanleyGoldman merged commit 6ad26d6 into github-for-unity:documentation-fixes Aug 27, 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.

2 participants