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

More simple and efficient #129

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

More simple and efficient #129

wants to merge 2 commits into from

Conversation

mhfgeek
Copy link

@mhfgeek mhfgeek commented Sep 30, 2020

I just make it more simple and clean, because too much code before.

@AceLewis
Copy link
Owner

AceLewis commented Oct 1, 2020

To be clear, this repo is literally a joke. The code is supposed to be bad, this is not how you would create a calculator.

The README.md explains why I made the joke, and the generator (generator.py) that made my_first_calculator.py is in the repo.

I have suddenly got a lot of PRs probably because Hacktoberfest has started. I am going to copy and paste this comment on all PRs to save my time. If you are trying to do Hacktoberfest please look elsewhere, DigitalOcean have issued an update to reduce the spam: https://hacktoberfest.digitalocean.com/hacktoberfest-update

https://blog.domenic.me/hacktoberfest/

@mhfgeek
Copy link
Author

mhfgeek commented Oct 2, 2020

Hahaha I am so sorry, I dont read the README.md

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.

2 participants