-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
effort: lowThe issue will probably take a single contributor no more than one day of work to resolve.The issue will probably take a single contributor no more than one day of work to resolve.meta: good first issueThe issue is especially suitable for first-time contributors.The issue is especially suitable for first-time contributors.priority: trivialThe issue has very little impact on people using the software.The issue has very little impact on people using the software.type: polishThe issue involves making existing functionality more usable.The issue involves making existing functionality more usable.
Description
Currently the file contains lots of things, like unknownLength/UnknownLengthSequence and makeUnidirectional/UnidirectionalSequence. These should each be moved into their own files inside the test directory and sequenceTools.js should be removed.
See #158 (comment)
To prevent existing code from breaking it will be necessary to also remove the sequenceTools import in higher.js and add an import for every file separated out in this way.
Metadata
Metadata
Assignees
Labels
effort: lowThe issue will probably take a single contributor no more than one day of work to resolve.The issue will probably take a single contributor no more than one day of work to resolve.meta: good first issueThe issue is especially suitable for first-time contributors.The issue is especially suitable for first-time contributors.priority: trivialThe issue has very little impact on people using the software.The issue has very little impact on people using the software.type: polishThe issue involves making existing functionality more usable.The issue involves making existing functionality more usable.