-
Notifications
You must be signed in to change notification settings - Fork 441
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
Conversation
@@ -0,0 +1,6 @@ | |||
# Sysroots |
There was a problem hiding this comment.
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
@@ -1,7 +1,7 @@ | |||
{ | |||
"name": "com.unity.multiplayer.mlapi", | |||
"displayName": "MLAPI Networking Library", | |||
"version": "0.0.1-preview.1", | |||
"version": "0.1.0", |
There was a problem hiding this comment.
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: |
There was a problem hiding this comment.
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
README.md
Outdated
|
||
Contents: | ||
* com.unity.multiplayer.mlapi -- the MLAPI SDK | ||
* com.unity.multiplayer.transport.utp -- the UTP transport library |
There was a problem hiding this comment.
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 :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixing
There was a problem hiding this 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?
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. |
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. |
Ok, I have moved copies of CODE_OF_CONDUCT and LICENSE and CONTRIBUTING to the root dir. |
MTT-476