-
Notifications
You must be signed in to change notification settings - Fork 107
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
Added basic analysis and introduction to TARS in README.md added a License.md #237
base: master
Are you sure you want to change the base?
Conversation
|
||
## Basic Layout Suggestion for your Analysis: | ||
- Topic Introduction | ||
- [Project/Topic basic information, links, etc] |
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.
Please give a proper file name - Not "README.md" (like tars-cloud-basic-analysis)
Analysis/GeneralTopics/License.md
Outdated
All rights reserved. | ||
|
||
Redistribution and use in source and binary forms, with or without | ||
modification, are permitted provided that the following conditions are met: |
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.
This file is not needed under GeneralTopics/ It gives a different meaning. If you want to mention the license of tars, you can mention in the analysis document itself
@skdwriting I have made the changes according to your comments, Kindly review the changed files and let me know if any further changes are required. Thank you. |
this software without specific prior written permission. | ||
|
||
|
||
## Contributing: |
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.
No relevant for our analysis. Or you can make it explicit to describe - how to contribute to tars. Otherwise, remove the section
| **Twitter** | https://twitter.com/TarsCloud | | ||
|
||
|
||
## License: |
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.
Please mention it is the license of what?
Also, please explain what is the license and why are adding it in the analysis
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #236
Test Report Added?:
Test Report:
Special notes for your reviewer: