New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Drop itertools deps #58

Merged

ManevilleF merged 2 commits into main from feat/drop_itertools

Mar 31, 2023

Owner

ManevilleF commented Mar 26, 2023 •

edited

The itertools dependency is not very useful

Work done

Cleaned Hex::wrap_with to use Vec instead of a temporary iterator in itertools
Removed deprecated niche method Hex::directions_to

ManevilleF added this to the 0.6.0 milestone

ManevilleF force-pushed the feat/drop_itertools branch from 34e577f to 1c184f2 Compare

March 26, 2023 13:34

ManevilleF added 2 commits

March 31, 2023 09:37


          Drop itertools deps

266061f


          wrap fix

e52108d

ManevilleF force-pushed the feat/drop_itertools branch from 20e1260 to e52108d Compare

March 31, 2023 07:37

ManevilleF marked this pull request as ready for review

March 31, 2023 07:39

ManevilleF merged commit 127b8b5 into main

8 checks passed

ManevilleF deleted the feat/drop_itertools branch

March 31, 2023 07:50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment