search

guardian / scribe-plugin-noting

9 stars 3 forks source link

Hg throttle mutation #26

Closed robinedman closed 9 years ago

robinedman commented 9 years ago

Throttle CPU intensive ensureNoteIntegrity function.

hmgibson23 commented 9 years ago

:+1: to easy performance wins

sihil commented 9 years ago

I would suggest that we pair on something then it should be a third pair of eyes reviewing the PR.

hmgibson23 commented 9 years ago

Makes sense.

On 11 November 2014 13:55, Simon Hildrew notifications@github.com wrote:

I would suggest that we pair on something then it should be a third pair of eyes reviewing the PR.

— Reply to this email directly or view it on GitHub https://github.com/guardian/scribe-plugin-noting/pull/26#issuecomment-62549364 .

Visit theguardian.com. On your mobile and tablet, download the Guardian iPhone and Android apps theguardian.com/guardianapp and our tablet editions theguardian.com/editions. Save up to 57% by subscribing to the Guardian and Observer - choose the papers you want and get full digital access. Visit subscribe.theguardian.com

This e-mail and all attachments are confidential and may also be privileged. If you are not the named recipient, please notify the sender and delete the e-mail and all attachments immediately. Do not disclose the contents to another person. You may not use the information for any purpose, or store, or copy, it in any way. Guardian News & Media Limited is not liable for any computer viruses or other material transmitted with or as part of this e-mail. You should employ virus checking software.

Guardian News & Media Limited is a member of Guardian Media Group plc. Registered Office: PO Box 68164, Kings Place, 90 York Way, London, N1P 2AP. Registered in England Number 908396