-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Used table format for README summary of Commands. #715
Conversation
Used table format for README summary of commands.
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 is looking good! What do you think about adding a third column to display default values?
Also set the text to left aligned.
Created Column for default values and set text to left aligned.
Fixed the column bug
Co-authored-by: Jade Michael Thornton <jademichael@jmthornton.net>
Co-authored-by: Jade Michael Thornton <jademichael@jmthornton.net>
Co-authored-by: Jade Michael Thornton <jademichael@jmthornton.net>
Co-authored-by: Jade Michael Thornton <jademichael@jmthornton.net>
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.
Hey, is there anything left to do in my PR? |
Was just waiting for the tests to pass! Looks all good! |
Please ensure that your pull request fulfills these requirements:
master
branchWhat is the purpose of this pull request? (bug fix, enhancement, new feature,...)
What changes did you make?
Provide some example code that this change will affect, if applicable:
Is there anything you'd like reviewers to focus on?
Please provide testing instructions, if applicable: