-
Notifications
You must be signed in to change notification settings - Fork 65
feat: Restructure README #598
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
feat: Restructure README #598
Conversation
|
Thanks for making a pull request! 😃 |
praveenj83
left a comment
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.
Multiple links are broken and the data processing section has not moved out. If any fix is more involved, I am ok to create a separate issue for it.
Signed-off-by: Dushyant Behl <[email protected]>
Signed-off-by: Dushyant Behl <[email protected]>
Co-authored-by: Praveen Jayachandran <[email protected]> Signed-off-by: Dushyant Behl <[email protected]>
45165d5 to
d04b5a2
Compare
Signed-off-by: Dushyant Behl <[email protected]>
d04b5a2 to
77a95b6
Compare
praveenj83
left a comment
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.
Some fixes were missed
c4e364c to
ae511a0
Compare
Co-authored-by: Praveen Jayachandran <[email protected]> Signed-off-by: Dushyant Behl <[email protected]> Signed-off-by: Dushyant Behl <[email protected]>
ae511a0 to
793d04e
Compare
Description of the change
Follwing the internal discussion we decided to restructure README into separate sections to make it smaller. This PR converts the README into three different pages making the main README smaller and more readable.
Was the PR tested
NA