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

Added more functions to calculator, updated UI, and created tests #69

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

naaimaqadeer
Copy link

No description provided.

@pH-7
Copy link
Owner

pH-7 commented Dec 6, 2021

@naaimaqadeer Thank you so much for your contribution! Would you mind giving me further details about your PR and the reason for it? Thank you!

@pH-7
Copy link
Owner

pH-7 commented Jan 1, 2022

Happy new year @naaimaqadeer! Anytime, if you coud let me know 😄
If not, I will be forced to reject your PR.

@pH-7
Copy link
Owner

pH-7 commented Aug 10, 2022

@naaimaqadeer Thank you so much for your contribution! Would you mind giving me further details about your PR and the reason for it? Thank you!

@naaimaqadeer Let me know if you have the change to get back to me 🙂

@pH-7
Copy link
Owner

pH-7 commented Aug 13, 2022

Let me know if you need any help @naaimaqadeer 🙂 Hopefully what I suggested makes sense to you 😊

@pH-7
Copy link
Owner

pH-7 commented Aug 20, 2022

Hey @naaimaqadeer 🙂 Are you are still willing to finish this and get your PR merged? If not, I will close it.

@LamprosGiotis
Copy link

Greetings,may i have my commit reviewed by someone?Thanks a lot!

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.

3 participants