-
Notifications
You must be signed in to change notification settings - Fork 82
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
initiated new directory for the documentation of linux notes #97
Conversation
added assets for storing images, etc... used for documentation
re-structuring the directory tree
added images
…r-linux-notes-inside-docs-directory
re-structuring directory tree
added assets folder for storing images and other files used in documentation
added /img/image.jpg
not needed
refined some text and added image reference from assets
Improved the definition of Linux Kernel
@bishal7679, would you please review my PR |
@Vrashabh-Sontakke sure! |
Please go ahead for the review if you want 🙂 |
…r-linux-notes-inside-docs-directory
Added linux_distro.jpg
added more
Okay @AvineshTripathi will look into it. |
…r-linux-notes-inside-docs-directory
Refractored
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.
Delete the last distro part a/q the suggestion
Co-authored-by: Bishal Das <[email protected]>
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.
LGTM!
@Vrashabh-Sontakke Thanks for contributing 🎊
Take a final look @AvineshTripathi
deleted accidental extra space Co-authored-by: Anurag <[email protected]>
…r-linux-notes-inside-docs-directory
Hey @kranurag7 , would you please review it now again. |
@kranurag7 this needs your review |
/lgtm |
hey @Vrashabh-Sontakke can you squash all the commits and merge into one so that I could fix the bug |
yes |
…r-linux-notes-inside-docs-directory
hey @AvineshTripathi , how do I squash merge an already open pull request |
follow this https://thoughtbot.com/blog/git-interactive-rebase-squash-amend-rewriting-history and then do a force push to the same branch Let me know if you need assistance |
@Vrashabh-Sontakke you want any help? |
@AvineshTripathi can you check the errors please ? |
I have tried solving the issue and 50% of work is done but there is some problem with |
No response from the original creator or the RP, closing this |
Fixes Issue
Closes #96
Changes proposed