-
Notifications
You must be signed in to change notification settings - Fork 0
Split functions and returned sequence types in test/sequenceTools.js into separate files #160
Copy link
Copy link
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.
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.
Currently the file contains lots of things, like
unknownLength/UnknownLengthSequenceandmakeUnidirectional/UnidirectionalSequence. These should each be moved into their own files inside thetestdirectory andsequenceTools.jsshould be removed.See #158 (comment)
To prevent existing code from breaking it will be necessary to also remove the
sequenceToolsimport inhigher.jsand add an import for every file separated out in this way.