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

Updating the ReadMe #313

Merged
merged 3 commits into from Aug 28, 2022
Merged

Conversation

krupalitrivedi
Copy link
Contributor

Reference to Issue no #310

Changes made in the file:

  1. License has been linked internally
  2. Minor Grammar Errors have been fixed
  3. Formatting has been done for consistency in the documentation

Necessary changes have been made, please let me know if you want me to make any changes after this one!

There are some sentences that I could see in which the rephrasing and grammar checking has been done. Please let me know if you find it giving out wrong message anywhere and I'd be happy to make changes again.
@yanivbh1 yanivbh1 mentioned this pull request Aug 28, 2022
Copy link
Contributor

@yanivbh1 yanivbh1 left a comment

Choose a reason for hiding this comment

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

@krupalitrivedi Thank you!
Would be great to have your thoughts about our docs as well in terms of organization, easy to read, missing information

@yanivbh1 yanivbh1 merged commit e21133e into superstreamlabs:master Aug 28, 2022
@krupalitrivedi krupalitrivedi deleted the changes-kt branch August 29, 2022 04:10
@krupalitrivedi
Copy link
Contributor Author

@krupalitrivedi Thank you!
Would be great to have your thoughts about our docs as well in terms of organization, easy to read, missing information

Sure! Will do that!

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.

None yet

2 participants