Skip to content

Comments

Editorial: rename internal run algorithm#246

Merged
annevk merged 2 commits intomasterfrom
annevk/process-all-the-things
Nov 2, 2020
Merged

Editorial: rename internal run algorithm#246
annevk merged 2 commits intomasterfrom
annevk/process-all-the-things

Conversation

@annevk
Copy link
Member

@annevk annevk commented Nov 2, 2020

"Run" is now "process a queue" and "process" is now "process an item".

Also tidy up the error mode note and shorten the "process an item" algorithm.

Closes #245.


Preview | Diff

"Run" is now "process a queue" and "process" is now "process an item".

Also tidy up the error mode note and shorten the "process an item" algorithm.

Closes #245.
@annevk annevk requested review from andreubotella and ricea November 2, 2020 11:48
Copy link
Member

@andreubotella andreubotella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@annevk annevk merged commit bf7ce77 into master Nov 2, 2020
@annevk annevk deleted the annevk/process-all-the-things branch November 2, 2020 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Editorial: consider another name for the run algorithm

2 participants