search

AmaniLouendriz / PontsDeConnaisssance

it's the website for a school support center
0 stars 0 forks source link

Testing branch #6

Closed AmaniLouendriz closed 2 years ago

AmaniLouendriz commented 2 years ago

Good morning @caseyli , @7chris71000 , I hope everything is going fine with you! Could you please review my pull request? Thank you so much, Amani

AmaniLouendriz commented 2 years ago

Hello @caseyli ! Thank you so much! I really appreciate it, It helped me a lot :) Commenting out code and not removing it is a very bad practice I have, and the names are more readable with dashes!

caseyli commented 2 years ago

@AmaniLouendriz looks great! And btw, commenting out code is not a "very bad practice", it's just that most of time you can confidently get rid of it because of Git.

The PR has been approved. You can merge it whenever!