Closed andeeliao closed 8 years ago
Hi @andeeliao,
There are a lot of spurious commits in this PR, including an extraneous merge commit. Please rebase this into a single commit. Single commits should include a full logical changeset with no extra diff information, so for instance there should be no record in these commits of you adding and then removing binary files, as that history adds noise.
I found a couple typos, otherwise LGTM!
If you would like to pair with me on rebasing your commits, I'd be happy to help out.
removed binary files from push