search

pearcej / opensource

Open Source: What it is and How to Contribute
4 stars 27 forks source link

Fix #152 by reducing the text in the symbol index list #292

Closed thaopham03 closed 9 months ago

thaopham03 commented 9 months ago

Fix #152 Reviewed by Moise

pearcej commented 9 months ago

@thaopham03 Please make sure this compiles correctly on your local. Then commit again to this same branch. And, post in the note the Github username of the person who reviewed your pull request.

thaopham03 commented 9 months ago

@thaopham03 Please make sure this compiles correctly on your local. Then commit again to this same branch. And, post in the note the Github username of the person who reviewed your pull request.

Hello Dr. Jan, I have fixed the issue and complied successfully in my local. Thank you so much!

pearcej commented 9 months ago

@thaopham03 Now you are committing 41 files which are mostly log files. Please update your branch and local then revert your last commit and then ONLY commit the single file that you changed!

pearcej commented 9 months ago

@thaopham03 please update your branch and then notify me

thaopham03 commented 9 months ago

I have updated my branch Dr. @pearcej

pearcej commented 9 months ago

@thaopham03 Please see the TA. This still has several errors and is not ready. I think you need assistance.

thaopham03 commented 9 months ago

Close this pull request as it appears to be too messy and not be able to merge onto the main.