-
Notifications
You must be signed in to change notification settings - Fork 46
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
Add documentation for the new Weekly InstructLab CLI Meeting #265
Conversation
Signed-off-by: Nathan Weinberg <nweinber@redhat.com>
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.
minor wording suggestion where it seemed like a couple words were missing
Co-authored-by: Russell Bryant <rbryant@redhat.com> Signed-off-by: Nathan Weinberg <31703736+nathan-weinberg@users.noreply.github.com>
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.
Noting @russellb's feedback on maximum line length linting for Markdown files - and this should be opened as a separate issue (if you would please do the honors, @nathan-weinberg) - this looks good to me.
Please rebase, @nathan-weinberg, so I can take one last look.
And noting to myself this should have been a review requesting changes and not a review with approval, which is my mistake. I am not certain how to fix that retroactively, but will investigate for future learnings.
@Mergifyio rebase |
✅ Nothing to do for rebase action |
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.
We seem to have lost @russellb's original wording change suggestions, so I would like him to take another look before we merge.
Co-authored-by: Leslie Hawthorn <lhawthor@redhat.com> Signed-off-by: Nathan Weinberg <nweinber@redhat.com>
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
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 too - merging.
Text is based off the existing text for the
Weekly All Community Meeting
but definitely open to suggestions!