Skip to content

Renaming proposal. #1

Open
Open
@snakeru

Description

@snakeru

Hi.

Having read the very same article and after some thinking I thought of implementing the very same library but luckily googled first. Congrats on making it before me :)

But I would suggest making the names shorter as the Go as a language tries to make everything concise and even the variable names are encouraged to be single letter in certain cases.

I observe the following problems (from my humble point of view) with the current naming (nursery.RunConcurrently()).

  1. The nursery term usually refers to raising children, but since we run our goroutines to their very death, it's probably not perfect.
  2. The RunConcurrently is already two words and just from the sound of it it's not clear, if the goroutines would be still running from after that
  3. The "nursery run parallel" is not really a phrase.

My ideas on how to name it would be:

  1. sync.Do(), though sync is taken so might cause problems for some people and still fails point (3)
  2. await.Run() - that seems to work around problem (2) due to package name choice
  3. scoped.Run() - feels a little better than previous one, tough the workaround for (2) is not there already.
  4. ...

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions