Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: cleaned up readme, 3rd party license, etc. #524

Merged
merged 6 commits into from
Mar 3, 2021

Conversation

mattwalsh-unity
Copy link
Contributor

@@ -0,0 +1,6 @@
# Sysroots
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Per guidance from @wackoisgod , the root folder just needs a README

@mattwalsh-unity mattwalsh-unity changed the base branch from develop to ci/validate-2021.1 March 3, 2021 03:02
@@ -1,7 +1,7 @@
{
"name": "com.unity.multiplayer.mlapi",
"displayName": "MLAPI Networking Library",
"version": "0.0.1-preview.1",
"version": "0.1.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ignore, this is coming from the ci fix

@@ -1,6 +1,17 @@
MIT License
This package contains third-party software components governed by the license(s) indicated below:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For some reason Git thinks this was a file move not a new third party file, probably because I deleted the LICENSE.md file from the root dir per @wackoisgod's guidance

@mattwalsh-unity mattwalsh-unity changed the title Docs/third party docs: update package manifest version to 0.1.0 Mar 3, 2021
@mattwalsh-unity mattwalsh-unity changed the title docs: update package manifest version to 0.1.0 docs: cleaned up readme, 3rd party license, etc. Mar 3, 2021
Base automatically changed from ci/validate-2021.1 to develop March 3, 2021 05:09
README.md Outdated

Contents:
* com.unity.multiplayer.mlapi -- the MLAPI SDK
* com.unity.multiplayer.transport.utp -- the UTP transport library
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: this isn't the UTP library - just a wrapper/interface to it :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixing

Copy link
Contributor

@JesseOlmer JesseOlmer left a comment

Choose a reason for hiding this comment

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

Moving these files into the mlapi sub-directory may be what the package standards demand, but it also partially breaks github norms - e.g. the auto-detection of the license for the repo. Any harm in having stuff like license and contributing in both places if it really is required in the package dir?

@wackoisgod
Copy link
Contributor

I guess there is no specific reason it couldn’t be replicated in the root other than maintenance. Though I did look at the graphics repo and does include a Lic file in the root.

@LukeStampfli
Copy link
Contributor

I think at least contributing and code of conduct should stay in the root? Like they don't have to be in the package and it will only confuse people downloading it? Readme is something we definitively want to at least have mirrored in the root.
Also the readme in master contains a bunch of useful stuff we could start from there to create a readme.

@mattwalsh-unity
Copy link
Contributor Author

Ok, I have moved copies of CODE_OF_CONDUCT and LICENSE and CONTRIBUTING to the root dir.

@mattwalsh-unity mattwalsh-unity enabled auto-merge (squash) March 3, 2021 23:36
@mattwalsh-unity mattwalsh-unity merged commit 399c7b3 into develop Mar 3, 2021
@mattwalsh-unity mattwalsh-unity deleted the docs/third_party branch March 3, 2021 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants