Closed ryanhugh closed 6 years ago
Just some more todos, from #29: Todo:
isRequired
argument in parsePreReqs
. We currently don't do anything with it, and the original goal was to add to optional prereqs if isRequired
was false, and required prereqs for if isRequired
was true.For your 3rd bullet, I don't think that's a good idea. I'd rather have it show up as 'None' rather than not showing up at all, because it shows that there isn't a pre/co req, rather than we have the lack of data to show it.
Truuuuuueeeeee. Lets do that
Note: I also fixed it when a class isn't a prerequisite for anything, it properly shows None instead of an empty line.
First bullet kinda done, the rest of the list is in reverse order and I can't seem to fix that. I'm calling it "good enough for now".
Are we pretty much done everything here? I'm working on some tests in the refactor branch and am down to hold off on tests until we get that branch in.
We're still missing documentation, but otherwise yeah!
Closing for now in favor of #15 and #35.