The content of the pull request looks good, and the tests are passing now. Before I merge the PR, however, you'll have to squash your commits into a single commit (and remove 170d9c4). In general, anytime you add a feature/fix a bug, the change should always be in a single commit. This means the vast majority of pull requests will have only 1 commit.
The content of the pull request looks good, and the tests are passing now. Before I merge the PR, however, you'll have to squash your commits into a single commit (and remove 170d9c4). In general, anytime you add a feature/fix a bug, the change should always be in a single commit. This means the vast majority of pull requests will have only 1 commit.