Skip to content

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