-
-
Notifications
You must be signed in to change notification settings - Fork 0
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.
I think this looks great π―
β¦o-course into feature/your-team
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.
@BekahHW sorry for the long reviews! But in some of them, I'm asking for your thoughts as well. π
This looks awesome and I love what you wrote here! I learn so many new things through this section and they're very helpful! π
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
β¦intainer-intro-course into feature/your-team
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
Co-authored-by: Ayu Adiati <[email protected]>
@adiati98 I think I've addressed all the comments. I really wish GH had a unified view for all the comments. It would actually be great to see them in VSCode so I know I'm getting everything. |
Yeah, that would be awesome. π |
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.
All are fixed. LGTM. Thank you, @BekahHW! π
Description
What type of PR is this? (check all applicable)
Related Tickets & Documents
closes #14
Mobile & Desktop Screenshots/Recordings
Steps to QA
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?