labofsahil/github-event-dataset-2012 huggingface dataset:
I've realized my site fails on line 509 of this module sometimes. I'd probably add the following snipped instead of removing unused import there. What do you think? I'm not happy with try/except but I'm not sure from where the wrong value comes. Maybe old user session ? 

```
try:
    products = products.order_by(sorting)
except FieldError:
   pass
```
Nova never really deletes rows from the database but uses a 'deleted' column in most of the tables. In this case the ``db.instance_destroy`` simply toggles the deleted column to on and all of the instance assessing functions filter on "deleted=0" in their queries. There are a few accessing functions where you can query all records in the database and we can use that get the instance and assert its state and terminated_at.

I'll use a similar pattern to checked terminated_at to test launched_at functionality in bless (and launch when those tests are added).
ups, misread that :)
This is from the copy pasta dinner we had for the `create_nova_client` above. I believe that was to make sure we added it to the context.
What the ... ?

Why not simply log with our logging framework and enable console via your own log4j.xml when you wanna see the console messages?
Oh, the processor FQN has changed... (again)
awesome, how did you find out?
Doubt it's more code. 

You have a try catch here which you'd not need at all and you trade that for keeping a future list and iterating it at the end.

The future list is equivalent to keeping track of correctnes AtomicBoolean + finished latch btw.

I definitely favour Callable for this type of work cos they're more readable and more suited for this type of work.

It's about using the right tool for the job, remember? :)
Dude - my name is everywhere!!
Couldn't find anything. I tried searching for:
 * [qt ppm](http://old.nabble.com/forum/Search.jtp?query=qt+ppm&local=y&forum=2903&daterange=0&startdate=&enddate=)
 * [qt png](http://old.nabble.com/forum/Search.jtp?query=qt+png&local=y&forum=2903&daterange=0&startdate=&enddate=)

The changeset which introduced it cee76c6ae16137a45b2ae1f29dfcdfb3ddfe4b0e seems to suggest it was just an extension rather than a bug fix. Since I was able to test this on my machine & it is consistent with the other image files, I suggest that this was worth highlighting, but not likely to cause any issues.
Is this supposed to be there?
property real imgRatio: isVideo ? 1.0 : implicitWidth / implicitHeight

and drop the imgWidth/imgHeight properties (as I don't see them reused elsewhere?)
The repetitive setup here is starting to grate on my nerves. :-)
Это для того чтобы сбрасывать у потомков если значения по умолчанию другие?
Jepp, Tomoya noticed this in the Chrome plug-in, so it got changed there - though even there subsites of oneliner.php aren't in the whitelist there.
We may want to bind this to something other then :server, Object#server seems a bit too generic, maybe Object#golitah_server?
This also needs a `height: 100%` or `min-height: 100%` to avoid repeating gradients.
Not really they all have a profile picture. If you do have an alternative can you attache it here?
@bp323 please remove the debug.log message
@gsilver Could you use 1 space between `.lhnavigation_data_container` and `.lhnavigation_private`?
Well stabbed ;)
i think it should be findOneOrNull or it would throw exception
Can't we get rid of the "defaults" option to set the format to json for every single action and rather set:
fos_rest.routing_loader.default_format: json
in app/config/config*yml ?
This is a common idiom for cases where the log message is expensive to compute. The idea is that you don't want to compute the whole log string just to throw it away because of the log level.
Don't think that's a major issue - but maybe it could be avoided by
defaulting to everything selected?
The idea was to separate the reports filters, from the data included.. I
started with just one 'select all' box but felt the behaviour was weird..
especially when unselecting it, I didn't expect it to deselect ALL the
options, and thought other would be confused too.


On Fri, Dec 7, 2012 at 8:24 PM, Angela Oduor <notifications@github.com>wrote:

> In application/views/admin/reports/download.php:
>
> > +					<td><?php print form::checkbox('data_include[]', '1', TRUE); ?><?php echo Kohana::lang('ui_main.include_location_information');?></td>
> > +				</tr>
> > +				<tr>
> > +				<td><?php print form::checkbox('data_include[]', '3', TRUE); ?><?php echo Kohana::lang('ui_main.include_categories');?></td>
> > +					<td><?php print form::checkbox('data_include[]','4',TRUE); ?><?php echo Kohana::lang('ui_main.include_latitude');?></td>
> > +				</tr>
> > +				
> > +				<tr>
> > +					<td><?php print form::checkbox('data_include[]','6',TRUE); ?><?php echo Kohana::lang('ui_main.include_custom_fields');?></td>
> > +					<td><?php print form::checkbox('data_include[]','5',TRUE); ?><?php echo Kohana::lang('ui_main.include_longitude');?></td>
> > +				</tr>
> > +				<tr>
> > +					<td><?php print form::checkbox('data_include[]','7',TRUE); ?><?php echo Kohana::lang('ui_main.include_personal_info');?></td>
> > +					<td></td>
> > +				</tr>
> > +			</table>
>
> When you get to the reports download page, for those who choose to select
> all, they now have to do that twice. Select all for approval status, and
> then select all for additional data. Makes sense?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ushahidi/Ushahidi_Web/pull/961/files#r2345580>.
>
>



-- 
Robbie Mackay

Software Developer, External Projects
Ushahidi Inc
m: +64 27 576 2243
e: robbie@ushahidi.com
skype: robbie.mackay
Another unrelated change -- just enhancing a test to make sure I didn't break the overwrite code in the version where I allowed setting cache-resident at run-time.   Even though I tossed that code, the test change is worth keeping.
Fixed. Concrete implementations all now refer the abstract class.
Этот файл уже лишний
Are you sure that ``$iterator`` (of type "object<RecursiveIteratorIterator>|object<RecursiveArrayIterator>") is traversable?
Why not just `this._events = {}` ?
You're welcome, and thanks for the credit.
Why complicate it? If the user wants to use a specific folder still can do it. I think this should remain as simple as possible. What kind of issues are you thinking on?
IANAL, but does the version log have any legal weight?
Well in my opinion if user is loged with facebook or has linked account (in future), he can change only selected attributes. First and last name should be same as on facebook. So that user should have custom form.

That is only my opinion ofcourse, I can change it (delete it) if u want it ?
Does not this throw a `KeyError` if dependency is missing? Probably it should be `cleaned_data.get(dependency, False)`.
(Changed above code a bit.)
It's hard to explain in just few lines. There is one _default_ connection (this is the one we are monitoring heavily) - think of it as status analogy of IRC. To join other channels some command must be send beforehand declaring "I want to join #foo". The same with unsubscribing "I want to leave #foo".  Just comment on that and I will do what you say next.
This should be cleaned.
The variable names different from the comment above and UDF declaration.
Done - we already call this function `get_app_info`, which does this.
I believe it is deleted and GitHub is just not showing it clearly. The file no longer exists in the branch I pushed.
No need to store jqxhr unless you're going to use it so just `$.ajax('hello/json/' + name, {` is good enough for that line
no individual ID's on these links.
If comment above is applied, this can be stripped
You can use directly the range builtin
BTW that _net.dataforte.cassandra.pool.DataSource_ is just a thin wrapper around _org.apache.cassandra.thrift_, which seems to be the most used / stable client, maintained by the Cassandra team.
This redirects to login.anosrep.org but this being master I think we should even point to dev: login.dev.anosrep.org
There are lots of repeated terms floating around, let's be clear and use `resolveDeferred` and `rejectDeferred` here.
Ok, that's old code though, but keep it for consistency.
`JSON.stringify(tpl);` is quick! Real quick! I could barely profile it with a nested template around 200 k. Now with the flat list the template is much smaller.

Correct. We can `return` right away No need to fiddle with the scrollTop and scrollLeft in that case.
This is IMO easier to read with constants, like `REPLACE` and `NO_CACHE`, then you don't need the comments.
This looks ugly IMO, I would just spell it out: `global_handler`.
Not used, as noted above.
You are right, mostly it doesn't. Planet with albedo = 0 should be basically iceball. It can happen if the planet had thick atmosphere which froze, but planets with thin atmosphere should have higher albedo. I think I should probably dive more into this. I will close this PR and to the changes properly (in collaboration with @Ae-2222 and @skapusniak). (Or what is better for the people doing the merging? Lot of issues with small commits which make sense and are easy to judge, or bigger "all in one" commits?)
that method work alright, I actually used that method later to resolve the library once I figured out which one to be included.

The problem is that libusb used this syntax to load library:

```python
def _load_library():
    candidates = ('usb-1.0', 'libusb-1.0', 'usb')
    for candidate in candidates:
        libname = ctypes.util.find_library(candidate)
        if libname is not None: break
    else:
        # ...
    # Windows backend uses stdcall calling convention
    if sys.platform == 'win32':
        l = WinDLL(libname)
    else:
        l = CDLL(libname)
    # ...
    return l
```

The filename given in CDLL or WinDLL is not a **constant**, so the code scanner in pyinstaller can't figured out what has been passed in. As for fixes, I don't see how this can be easily fixed. It may require a complex static analysis to make sure things go right in my opinion.
Constraints is misspelled here
Why capitalized again? I believe it [was fixed](https://github.com/svenfuchs/ruby-cldr/pull/9) in `ruby-cldr` to be consistent with other components, e.g., `numbers`. Same with `calendars` below. I think you should remove this capitalization both here and in your `ruby-cldr` fork.
How about a protected method to return a new connection instance, e.g.:

```php
<?php
    /**
     * Returns a new connection for a given URL. Override for testing purposes,
     * for example.
     *
     * @param  string url
     * @return  peer.http.HttpConnection
    protected function connectionTo($url) {
      return new HttpConnection($url);
    }
?>
```
It's the alpha for the white rectangle drawn on the overlay canvas, but the visible effect is to apply a (1 - alpha) fading effect to the graph canvas content, taking it closer to white. Let me know if you think it would be clearer to reverse it with default 1 and referring to it as background alpha. Maybe not quite accurate but less confusing?
Please move choices to the top-level constant.
actually I should fix this and the spacing so the diff isnt so heinous
I don't this would be needed if we bound to @model.on('error', @displayValidationErrors) either.
Just pushed an edit where this style is also adopted in the later discussion of the shear class in isolation (S6)
this file shouldn't be in there right?
Can we do a stronger assertion here? It looks like it would pass even if there were no entities returned?
We'll need a version of this scrip that will work with the `.tar.gz` distro (ie.g. classpath is lib/*.jar, because there is no "target" directory in the distro.

But let's look at that in a subsequent pull request.
This one also bothers me. true/false is the standard in modern languages. There doesn't seem to be a semantic benefit to using no over false, it just makes the code harder to read.
this is a required field here which means even if someone dropped their tables and migrated their data into this schema, they would not be able to since this has no default.
Please do not remove the request.validated[attr.name] = attr.missing, it's part of the API and shound not be changed. Is there any reason why you're removing it?
But isn't that the environment were the script runs?
For some reason, things like assert_in_delta weren't available, possibly because of the way I ran the tests without rake.
you should remove the extra space after the function name
"the pleasure to host" sounds a bit stiff to my ear. How about "the pleasure of hosting"? :)
Same question as above: should we use ``CommunicationEventType``s and ``Dispatcher`` here?
you probably want to define MissingProductImage somewhere
Well the value would either true or false so it should be fine either way. The ca is a Boolean and ca_server is the actual value.  


On Friday, August 24, 2012 at 5:12 PM, Greg Sutcliffe wrote:

> In templates/server/puppet.conf.erb:
> > @@ -4,6 +4,7 @@ > reports = foreman > external_nodes = /etc/puppet/node.rb > node_terminus = exec > + ca = <%= scope.lookupvar("puppet::server::ca") %>  
> that would leave you with "ca = ". Would it not be better to do something like:
> <% if scope.lookupvar("puppet::server::ca") -%> ca = scope.lookupvar("puppet::server::ca") <% end -%>  
> which would omit the line entirely?
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/theforeman/puppet-puppet/pull/9/files#r1457649).  
>  
>  
>
Hi Austin,

Have you checked this doesnt' change the size of the pat tuples significantly?

Evan
Admin must be of type User.
This one needs to be changed into: cats.push(cat.value);
Your version generates a javascript error and causes the spinning arrow (job count) to stay active.
Also, ModeShape will never use this INSTANCE, since it always constructs its own instance.
Why not ```belongs_to :user``` here?
Надо делать человекопонятные названия полей.
Это я скопировала к себе для тестирования времени. Видимо случайно за коммитилось.
Надо тайм-зону устанавливать у SimpleDateFormat
I'm thinking that instead of committing all this commented out code, we could open a JIRA and put the code on the ticket.
There might be, lets get his input.
Well, it's only going to get more confusing as the current 'api' profile will be split into:

 * drush
 * Drupal 6
 * Drupal 7
 * Drupal 8

so I'm not sure such a complex `behat.yml` services anybody as a starting point for their project.
I actually thought I was doing that. That's what's happening at https://github.com/mozilla/marketplace-tests/pull/45/files#L1R190 for example. Do you mean something else?
Use emdash or make a separate sentence.
FactorAnalysis sounds better to me :)
Same as above: space before ':'.
Does this test need some equality tests too or are those well covered elsewhere?  I took a glance at the full file, but it's organized by bug more than by functionality so gave up looking.
These should be with prefix underscore as agreed.
the problem is that sometimes two requests would create the same user, triggering save twice. so we just bail out here if that happened.
Whoops, yes. I'll revert 'tail' movement.
Should there be width specifiers to prevent buffer overflows?
Move it closer to use, or at the start so common with the other calls.  Should make it const.
My bad, I thought ```gitRepositoryFileHook``` was an object, not a pointer.
Is this just to yield other threads?
Proper licence here please.
Setting ``$wgBugzillaCacheType`` to ``dummy`` is the equivalent of setting ``$wgBugzillaUseCache`` to ``false`` in the previous version? If so, can you please add a comment saying that?
remove this and ...
Can we somehow avoid repeating all these constraints here?
add a blank line after `use`s block
I think this is actually wrong behavior. If `check_groups` is True, don't we want to check groups AND the user? Could use a unit test.
Nice cleanup on the line length tests. I agree that this should be a warning.
tu devrais rajouter :test dans le groupe

group :development, :test do
...
The shift is incorrect. You probably misused either tab or space
I overwrite these so that we can user newer versions than the one in the Maven 3.0 super pom.
@shacker can we remove the `console.log` here?
It might be better to use a URI template here: `/notifications{?all,participating,since}
add comment for null param
Were actually not really needed, removed altogether.
There's a typo, but don't worry, i'll fix it (start)
No newline can occasionally screw up SVN versioning for SVN clients.
a) Running this on a DC the railgun method inspects the network for domain controllers and wont discover itself.
b) Running on a network with 100 different domains it takes a LONG time to run and the majority of DCs dont allow access to someone not directly in their domain or trust.
c) You may only have scope to test specific domains on a job.
True. Do you think the new version is better? Or is there a simple way that I'm overlooking?
Yes.  This will help with issue 9.
If this isn't actually used, remove it
We could also move the polygon so the default camera view is OK.
Windows isn't fully POSIX complaint and for it we have win32.mak, as I understand.
Like I said, ignored words, preferred words and the scoring that comes with it ;)
We may want a null check here since we are calling toString() on the result of TiUrl.getCleanUri() which can be null.
Missing cap.

It seems like you were trying to start with a  capital letter, so I'm going to flag these for the sake of consistency, even though I don't think they're a big deal in a log message.
This code should be conditional on the following check:

```
if 'plist' in self.ios:
```

I believe this works even if `self.ios` is `None`.
Seems like these methods should not be triggered until after the current view appears/disappears, or there could be issues where a view thinks its parent is visible before it's on screen.  Probably should behave like `TiUIWindow` below.
Also, this should be a check after the isinstance.  If I am a list that already has data and one of those pieces of data is this embedded_list_document I just clobbered the whole list.  :(
Exception used to signify that an application element (class, webservice, method) does not exist
This should be testing for a bi-directional range, not an input range.
I see, thanks. I find it very arcane to rely on this kind of trickery to distinguish between types with disabled default constructor and the others. Any other ideas for a simpler way?
The current implementation requires length, so adding the restraint is correct. However, Chunks can function perfectly well on a slice-able infinite range, so it would be better (in another pull request), to improve Chunk's implementation.

I can do that.
Could `extern(Windows)`be factored out and used as a scope here?
This is wrong since it works on API 7+
Why **Executable**ParametersValidatable? Wouldn't ParametersValidatable be better
s/java/Java
accountManager, not context.
can these init functions be folded together somehow? seems like a lot of duplicated code...
What database is this written against? In MySQL you would need to use `BINARY ${field}=?` to do a case-sensitive comparison.
^ Read above. With my format, you can `each` through the array and process each segment individually.
or should we just remove ``getHeaders()``?
btw, the testsuite would fail because of this if you had tried to run it
Again, not deemed valuable (either available type was not specific, or it is inferrable from a type hint).
**Removed the code which writes the feeds into a file**
Also, the hash must be considered secure
А вот тут все ништяк с порядком инклюдов.
This Configuration class was deprecated in Infinispan 5.1 (and will be dropped in infinispan 6.0) and replace with the nicer org.infinispan.configuration.cache.Configuration. The total order related configurations shoudl be moved there and removed from here.
I would like to be consistent in how we do these internal module names.  Generally one of two styles is preferred:

    import matplotlib.patches as patches

or:

    import matplotlib.patches as mpatches

Use the latter if you are worried about local name clashes.
Exists of 2 extra functions now that handle the upload and association to a collection.
I didn't realize we had any changes in this file -- is there any reason they're in here and not our sakai base css?
changing the way links are rendered
@amolnayak311, in some PR @garyrussell made a note about empty @params in the JavaDoc:
0r just remove it, or make some description
Hi, Oleg!

We have a lot of tests with similar comment throughout the project. But how about if some day we decide to change `HttpRequestExecutingMessageHandler#onInit()` once more and this test will pass again. But it won't to reflect the new logic of our new changes...
And also I remember once you taught me to add at least one `assert` to tests.
So, where is the truth? 
What is the best practice to avoid test's confusing in the future?

Sorry, if I'm a bit sharp
It look like a bug in the spring-test project.
In the context of this test I use the same embedded-database config as @ghillert in the StoredProcOutboundGatewayWithSpringContextIntegrationTests-context.xml
My test runs first, inits in-memory DB and passes. But when StoredProcOutboundGatewayWithSpringContextIntegrationTests starts it is swearing that 'CONVERT_STRING_TO_UPPER_CASE' object already exists. Gunnar, let me guess ;-): that's why you are using different beans' ids in your tests contexts:StoredProcOutboundGatewayWithNamespaceIntegrationTests-context.xml & StoredProcOutboundGatewayWithSpringContextIntegrationTests-context.xml
Great! Thanks a lot, guys! Live and learn... I about SmartLifecycle.
However if you look into ```reschedulePersistedMessagesOnStartup()``` and his JavaDoc, I schedule polled messages on the initializing, don't process them.
And about 'dedicated scheduler is REQUIRED'. Maybe will it be enough to hide 'taskScheduler' creation inside DelayHandler?
Then we shouldn't worry about shared 'waitForTasksToCompleteOnShutdown' and simply manipulate him via SmartLifecycle...
However we still should care about 'MessagePublishingErrorHandler' for delayer's 'taskScheduler'. 
I don't know: maybe it also make sense to add ability about 'discard-channel' for problems with scheduled tasks...

Nevertheless, I agree about pushing this issue to M3. I need more time to think about your comments.
I see your point. I'll add a custom name to our configuration.
You changed the color from #f2f2f2 to #eff3f3.
4 spaces too much ;)
Adding class definition that inherits ActiveRecord in migration is not such good idea. In migration EntityType class is used to call constants, but it shouldn't be done this way. This is just a note.
Why are you not doing a jQuery extension? But modifies window?
Per above, if we can get the external files under NuSpec we need to explicitly specify the files element?
GitHub seems to have eaten a comment... 2nd attempt.
I tried this set of commands, it seems to have killed my config:
```
$ OMERO_CONFIG=another bin/omero config get
Traceback (most recent call last):
  File "bin/omero", line 123, in <module>
    rv = omero.cli.argv()
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/cli.py", line 1192, in argv
    cli.invoke(args[1:])
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/cli.py", line 742, in invoke
    stop = self.onecmd(line, previous_args)
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/cli.py", line 811, in onecmd
    self.execute(line, previous_args)
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/cli.py", line 891, in execute
    args.func(args)
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/plugins/prefs.py", line 65, in open_and_close_config
    config = self.open_config(argp)
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/plugins/prefs.py", line 138, in open_config
    return ConfigXml(str(cfg_xml))
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/config.py", line 108, in __init__
    self.version_check()
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/config.py", line 149, in version_check
    self.version_fix(v, version)
  File "/Users/simon/work/openmicroscopy/dist/lib/python/omero/config.py", line 171, in version_fix
    raise exceptions.Exception("Version mismatch: %s has %s" % (props.get("id"), version))
Exception: Version mismatch: another has None
```
Is this a bug in the CLI or incorrect docs? It also prevents any other config being read, `OMERO_CONFIG=default bin/omero config get` and `bin/omero config get` now return the same error.
Может как-то попроще перефразировать?
These `if`s read more nicely when extracted to private methods.
不要触碰与本次commit要解决的问题无关的地方。在这种触碰毫无道理的情况下更是如此。假如原代码有问题，要做成另一个commit
It does? I thought it used te vz stuff to find the ip.

Sent from my digital shackles. 

On Apr 5, 2012, at 10:34 AM, Tim Simpson<reply@reply.github.com> wrote:

>>     def test_multi_nic(self):
>>         """
>>         Multinic - Verify that nics as specified in the database are created
>>         in the guest
>>         """
>> +        if test_config.values['openvz_disabled']:
> 
> This checks nics in the actual DB, so it could just use:
> <pre><code>
> if WHITEBOX:
> </code></pre>
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite-integration/pull/38/files#r650731
Ok...

I changed the code from

        if (strEventId === "payment.purchase.callback") {
            _event.trigger("payment.purchase.callback", JSON.parse(args));
        }

to 

        if (strEventId === "payment.purchase.callback") {
        	stopEvents();
            _event.trigger("payment.purchase.callback", JSON.parse(args));
        }

That way the code will stop event (from the Native side) and then trigger the event to the client side. Hopefully that will work.
Pero si se llama Perez Roberto, estás en el horno.
чтобы
Missing space: propertiesin => properties in

Also, for the platform name in text, we've been using Mobile Web (with a space).
animate: is exposed to JS,  do we allow users override it ?
Class: java.util.HashMap
classFilter: *.class = true
includeFilter: org.jboss.* = false

All the filters are then composed into an AndFilter, which will in the end evaluate to false in the above example. I.e. the include filter will exclude any classes that don't satisfy its path expressions.
That was the source of the idea of the puzzler. I found such an example in a column by Venkat Subramaniam (http://pragprog.com/magazines/2012-02/scala-for-the-intrigued) where he defined a class
```scala
class Resource private {
  ....
}
```
and I asked me what does that mean? So I find it funny that with ohly the keyword private a private constructor is defined, in particular even if you don't add paranthesis. 

The first version of the puzzler actually was
```scala
class C1 private 
private class C2
```
but this one requires a semicolon. Semicolon inference does not work here.
Regarding "questionable Scala style" one should probably make a comment in the explanation section (but the Scala style guide does not address this point).

I could live however with the following version:
```scala
class C1 private {}
private class C2 {}
```
here semicolon inference works. One could also add `/* ... */` in the body of the class.
he should discuss that with Alec. It was his idea.
Why don't your DataContract and class have the same name?
Doesn't this mean we allocate a string on every "turn_into_nop"?
Brilliant! I search for that dumb artifact for like 10 minutes and couldn't find it. I concluded they didn't publish it. That's why I ended up with the antrun plugin. Nice find.
This should use brackets.
The check must be done in the DependencyInjection extension. So there is not check at runtime
Nevermind, forgot we were in the packager
"css_class" was introduced into Collection portlet starting from Plone 4.0.x. This change will break Plone 3.x compatibility.
dispatch event takes an array I think? right?
This should be removed since it was added in a separate pull request.
@miwong Yes it is. I have been manually adding it for our SCM builds. FYI, I have also added the  "-lang-c++" flag for the connection API in pull req 63 [https://github.com/blackberry-webworks/BB10-WebWorks-Framework/pull/63]
log out session in crowd
Nope, it's only a break for someone using `render_model` directly.  Such as [armstrong.hatband].

[armstrong.hatband]: https://github.com/armstrong/armstrong.hatband/blob/master/armstrong/hatband/views/search.py#L110-114
Did you check that this runs with jasmine-node lastest? Anything after 1.0.7 breaks the CLI like mad. The only solution is to write another one, or change the specs to look like how it expects it (last time I checked). :-(