New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
improve notification routing doc 2 #22
improve notification routing doc 2 #22
Conversation
@ali-graham and @Hobbsee your feedback on this would be appreciated |
Hmm, I hadn't seen Github's image diff-er before... nice! Yup, looks good. |
2. Run this shell: | ||
1. **Receive events.** | ||
* Events are generated by external check execution systems and created as json objects in the `events` queue in Redis. | ||
* Processor removes events off the queue and, astonishingly, processes them one by one. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tense on this line doesn't match the rest of the document, and astonishingly probably isn't needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's current tense, like the rest of the document. am I missing something?
That's a really nice gif, and I learned some stuff around this today. Thanks, @necula01 & @jessereynolds! |
…_routing_doc improve notification routing doc 2
Same as #21 but with some additional commits.