-
Notifications
You must be signed in to change notification settings - Fork 132
Reorganizing the headers #1689
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
Reorganizing the headers #1689
Conversation
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.
Eagerly waiting to see the new layout!.
Co-authored-by: Revathy Venugopal <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #1689 +/- ##
==========================================
+ Coverage 85.23% 85.27% +0.03%
==========================================
Files 45 45
Lines 7506 7506
==========================================
+ Hits 6398 6401 +3
+ Misses 1108 1105 -3 |
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 all looks good to me! Thanks!
Thank you a lot Kathy! Co-authored-by: Kathy Pippert <[email protected]>
Regarding #1688 I have decided to NOT join both sections. The WSL section is more specific than docker and it has its complexities. |
Co-authored-by: Kathy Pippert <[email protected]>
By popular demand.
@PipKat although there is some renaming, I did not change anything in API directory. Have a look at the other files (docker, wsl, etc)
I'm merging tonight so if you cannot have a look today, feel free to open a later PR.
@Revathyvenugopal162 ... Finally you got it! xD
Close #1688
Close #1620