Closed parkerl closed 7 years ago
Looks good to me. Is this based on this style guide? (also, doesn't removing the plunit library break the tests?)
Edit: it doesn't break the tests. Not sure why I was convinced it needed to be there.
Great. Yes it is based on that and on past experience. If you can get the other PRs fixed up we will get them all merged. We are off to a go start.
@qjd2413 here are some style changes I'd like to suggest based on "leap". Let's use this PR to discuss.