-
Notifications
You must be signed in to change notification settings - Fork 149
chore: changed the visual look, added emojis, and edited grammar. #159
Conversation
Thanks for opening this pull request! Please check out our contributing guidelines. |
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.
Hi @NoorinSakhi,
Everything is good, but please remove all [ ]
to all files
I removed all the |
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.
Hi @NoorinSakhi,
Sorry I didn't see you are converting the text to tile, and it doesn't look very well, so my recommendation add the emojis and edited the grammar
Kudos, SonarCloud Quality Gate passed! |
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.
Hi @NoorinSakhi,
Nice work
Congrats on merging your first pull request! We here at MakeContributions are proud of you! 🎉🎉🎉🎉. |
Please take a look at my changes and see if you approve thank you.