RyanEdwardHall / anagrambler

Finds all anagrams and sub-anagrams in a string
MIT License
4 stars 1 forks source link

Add module file and makefile #20

Closed mcgid closed 2 years ago

mcgid commented 3 years ago

No more GOPATH, finally. This also adds a makefile for our convenience, and fixes some linting and a deprecation.

Easy review mode: read the commits in sequence, starting here. Should be easier to follow.

As I say in the commit, I decided to move the implemenation into the internal directory because that avoids the public contract, and also keeps the root a bit cleaner.

The tests are still broken. I realized that the test fixture data is tightly coupled to the dictionary data, and as such the tests aren't unit tests. I hope to fix that at some point.