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

Missing Exercises #226

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

Conversation

fyfynguyen
Copy link
Contributor

@fyfynguyen fyfynguyen commented Dec 17, 2018

Hi Chris,

Last few completed assignments are included in this PR. This should put me right at the 80% completed...

  • Html render: this one was a beast. So hard. I did my best but its current state is kind of a hot mess. I haven't quite gotten all the tests to pass and everything working right. Everything seems to be failing more the more I refactor... I need to keep working on this one but likely won't be be able to push any updates before you submit.

  • circle.py: after doing html render... I didn't think it was possible to see green and pass all tests! Got it working though and all tests pass.

@PythonCHB
Copy link
Contributor

FyFy -- yup, html_render is challenging -- that's why it's so many points.

Taking a look ....

@PythonCHB
Copy link
Contributor

you've accidentally added pycache directories -- be careful!

I think that happened because you deleted the .gitignore.

but also -- you need to be careful with git add -- not to add stuff you don't want.

@PythonCHB
Copy link
Contributor

giving you full credit to html_render, which brings you JUST 20 80%.

But you really want to do OO mailroom and html_render to make sure you understand OO programming.

I also don't see a trigrams -- that's another good one -- maybe go back and give it a try again -- you may be surprised, now that you have a bit more experience.

@PythonCHB
Copy link
Contributor

not merging, due to git issues -- and class is over anyway :-)

@PythonCHB PythonCHB added the git Problem not merging due to git issue label Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
git Problem not merging due to git issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants