Thanks to visit codestin.com
Credit goes to github.com

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

Update getting-started.md #893

Merged

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