Skip to content
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

Mentor guide #1024

Merged
merged 4 commits into from
Jun 21, 2023
Merged

Mentor guide #1024

merged 4 commits into from
Jun 21, 2023

Conversation

aliok
Copy link
Member

@aliok aliok commented Jun 15, 2023

Fixes: #736

I've written the mentor guide here: https://docs.google.com/document/d/1Zozg3g5qE4jkdVEdiVtCoXvc_iuuOls1hfCgW6Pf7jQ/edit#heading=h.vpidi5cz3gv4

Happy to collect feedback there.

Later on, I will use this PR to update the markdown doc.

UPDATE: I've updated the doc with the content from the Google doc mentioned above

Signed-off-by: Ali Ok <aliok@redhat.com>
@aliok aliok changed the title Update README.md Mentor guide Jun 15, 2023
@aliok
Copy link
Member Author

aliok commented Jun 15, 2023

Please review @nate-double-u @jaytiaki @Riaankl @CathPag @jberkus

@riaankleinhans
Copy link
Collaborator

riaankleinhans commented Jun 15, 2023

Added a review. Great work Ali.

@jaytiaki
Copy link
Collaborator

Awesome work @aliok! I've added a review too

@CathPag
Copy link
Collaborator

CathPag commented Jun 16, 2023

This awesome, @aliok! 💙

Copy link
Collaborator

@jaytiaki jaytiaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

aliok added 3 commits June 20, 2023 00:32
Signed-off-by: Ali Ok <aliok@redhat.com>
Signed-off-by: Ali Ok <aliok@redhat.com>
Signed-off-by: Ali Ok <aliok@redhat.com>
@aliok
Copy link
Member Author

aliok commented Jun 19, 2023

@jaytiaki @nate-double-u

I've updated the doc with the content from the Google doc mentioned above.

Ready for final approval

@aliok aliok mentioned this pull request Jun 19, 2023
Comment on lines +69 to 72
## Success Stories

## FAQs
TBD

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Created #1025 for this draft section. I would like to merge this PR regardless.

Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this @aliok!

@nate-double-u nate-double-u merged commit e481997 into main Jun 21, 2023
@nate-double-u nate-double-u deleted the aliok-patch-3 branch June 21, 2023 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Next steps for Mentor section
5 participants