Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

AlgPart constructors require new, ____SiGNMove helpers cannot be called with new #20

Open
lgarron opened this issue Jul 15, 2018 · 0 comments

Comments

@lgarron
Copy link
Member

lgarron commented Jul 15, 2018

Example:

new Sequence([BareSiGNMove("R")])

It's probably confusing to beginners to know when to use new. TypeScript and ES6 require new for classes, and I don't want to create a helper for every class. So the main workaround I see here is to make SiGNMove an abstract class, and SiGNMove helpers into actual subclasses of SiGNMove.

@lgarron lgarron changed the title AlgPart constructors require new, *SiGNMove helpers cannot be called with new AlgPart constructors require new, ____SiGNMove helpers cannot be called with new Jul 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant