Rename async to something else
See original GitHub issuePlease define your problem or issue as clear as possible.
Vivus version: 0.3.2
Browser: all
I saw this project posted on /r/javascript. I just wanted to start a dialogue regarding the async
method. I think it’s confusingly named. I know that in programming synchronous
means to do things one after another, while async
means, well, to do them as things happen (via callbacks or using promises or async
/await
in JS).
But when I think about what the drawing method that is used when calling async
in this library, it reminds me of marching bands and synchronized swimming and such. The lines are drawn in a synchronized fashion. So I feel like async
is a confusing name.
I’m not here to crap on this project or anything, I just thought it’d be good start a dialogue on this before the project hits 1.0 and the name can’t easily be changed.
Thoughts?
Issue Analytics
- State:
- Created 7 years ago
- Comments:9 (4 by maintainers)
Top GitHub Comments
Dammit! I wanted to discretely pass the
nsync
in the types list. But you got the idea before… let’s put it anyway 😃 About the naming, I saw something on Reddit I quite like : parallel. It’s simple and really describe my vision of it. Do you find it alright? I just want to stay away from ‘sync’.I wanted to keep ‘delayed’, and maybe rename the ‘oneByOne’ to something like ‘queue’.
I’ll list some options. Hopefully some other people will get in here and throw a ruckus about how bad my suggestions are.
sync
synchronized
atTheSameTime
allTogether
justinTimberlake
(get it, b/c everything is drawn “n’Sync”)Ok, so that last one is a joke, and the other ones all have flaws. But if we’ve got
OneByOne
thenAtTheSameTime
might work.As far as the other names, I think they’re fine, but that’s just, like, my opinion, man.