- Crack method successfully implemented, but with brute force
- CLI successfully implemented
- Classes are well-organized with single responsibilities
- Apply_shift and remove_shift are a little long and repetitive
- Variables and methods have clear names
- No class methods
- 100% coverage as shown by simplecov
- Stubs are used to test random strings
- Appropriate amount of commits and PRs
- Naming and description of commits is slightly ambiguous
- Most commits are small chunks of functionality