labofsahil/github-event-dataset-2012 huggingface dataset:
We really need to cache this.  And also, do you really need the full concept back?  Why not skip that extra database query and just give back the concept id?
modify synapseGet to take an optional argument specifying whether or not to check the curl response (see "checkCurlHandle"). Then pass your own curl handle to synapseGet and extract the return code directly.
I'll add another test. 
As for the other thing I would not worry about it as much. First its a very corner case, second there is no harm in payload map being replicated as headers (other than space  may be), so adding a new attribute now would be an overkill IMHO.

Thoughts?
May be but let's talk about it. Regex might also be a bit heavier and if it could be avoided why not
Don't change the version. Doing releases is separate from functionality changes.
Can you make a method in headers.rb that does this? This functionality is the mirror of env_to_headers, i.e., headers_to_env.
@apex13 Shouldn't this be `Summary`?
Can you use the ``JsonResponse`` object and fix CS ?
If you're polling a source every 30 seconds, you'd want the objects with the same id's to be updated with the latest info including time information.
You should add a paragraph explaining that the Js code triggers some events
good catch, please pull out as constant
This would make this unusable by people not using Grappelli. If Grapelli changes the id of the search field, the code must handle both cases.
So who's to make the first move?

doctrine/doctrine-bundle (which only has one branch dev-master) requires doctrine/dbal: 2.2.*
Wouldn't it make sense for DoctrineBundle to create a 2.2.x branch and require the 2.2.* dependencies?
How about implementing `add-base64-auth-header` in terms of  `add-auth-header`?
Ok.  Will do.
There is another tricky way to restart the service and is by changing one of the AppSettings, rather than redeploy which as Tom mentioned, is a heavy operation. Do we have a plan to add this ability to the portal? Do the portal and the CLI be symetric in terms of API?
No, it also includes updates to the data sent back and forth. You were only sending useful data on spec results, I send data on spec, suite and runner results. Additionally I send when the runner starts.
webKit
Should this have been 4 spaces instead of a tab?
using good libraries that aren't jquery is a sign of someone we want to hire I am very strongly opposed to this requirement
Nah. If the test leaks file refs, it can die in a fire.
Thinking more about the long term - what would be the best way to store the actual run times and later on expose them over JMX or something?

I'm jut curious and this way we could see the actual values for `System.currentTimeMillis() + start`
Why this change? 

lazy_build => 1 already implies lazy => 1 and builder => '_build_author'
OK, I'll just push that change to master. I see that @karlcow merged this PR yesterday. :)
here you're using bold markup in the `@return`, but I haven't seen it in other places..
Do we want to move this into a com.gu.management.servlet package?  Seems a bit weird to have a package object with lots of servlet specific stuff defined at the top level.
Didn't know that...useful.
Minor thing, but could you mark this as a FIXME instead of a TODO please?
As mentioned above, give it more indention and switch to tabs, not spaces.

Also, `'{$starttime}'` is not recommended. Better to use:
```
SET starttime = '" . $starttime . "'
```
or at least:
```
SET starttime = '$starttime'
```
This time I'm pretty sure that's fixed.
Oh, I see it. That worked too..
looks thread safe to me
Makefile? auto generated?
The parameters in the documentation don't max the actual function
Those are just file permissions.  I think it's ok to set them per file, even though 0755 is generally ok.
No in 2.0 you should keep this js, the second one is only avaliable for 2.1