Skip to content

Conversation

@esc
Copy link
Member

@esc esc commented Apr 4, 2023

As title

kc611 and others added 30 commits March 26, 2023 23:32
Rename the variable that is bound to the outgoing edges and use the list
indexing to access the elements.
Adds the typing signatures to the NameGenerator, adds numpydoc
compatible documentation and does some refactoring on the code, mostly
variable names and shortening one construct.
add typing and refactor NameGenerator
We add some documentation, fixup the typing signature, refactor the
function to use return sorted lists, modify some implementation details
and add comments to the function. We return sorted lists, because we
have been bitten in the past by using sets and them being unordered.
This fixes two bugs on `main` introduced by:

* #26

and

* #28

It's probably a sign that the test coverage isn't high enough yet.
Formatting fixes and typos in README
Fix bugs leftover from refactoring
Added precommit configuration file
Adding documentation, slight internal refactor, change return signature
to be sorted lists. Also, refactored all uses of the function to account
for the change in return signature.
@esc esc marked this pull request as draft April 4, 2023 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants