Right; the point here is if they __explicitly__ set `cleanSession` to `false` then we must not unsubscribe.

Perhaps it is my phrasing here that's confusing - change it to

>Starting with _version 4.2.3_, the adapter will not unsubscribe (by default) if the `cleanSession` property is `false`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8361](https://teamcity.mesosphere.io/viewLog.html?buildId=87702&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:05:18


Why not use hash_combine() here?

This particular hash combination always yields 0 when the first and second values are the same.


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 3](https://teamcity.labkey.org/viewLog.html?buildId=289988&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:00


surely just remove this


To download the toolbox and the script within the same folder, please download the following ZIP


@wrxtasy did you do a clean build?


Да, ты прав, тут логично сделать дефолтным True, а в самом хэндлере по дефолту будет False (чтобы по умолчанию не сжимать, пока пользователь не попросил).


Yeah. Sure! Will change, also it is mine omission. You say about `cleanSession` default in the previous sentence.
Looks like that's all.
I can merge it with this simple polishing :smile:


> IIRC I was getting AI errors after I renamed the colony base hull

In the most recent version or in your previous unfinished approach? Seems like something that should not happen anymore (in fact I can't reproduce it). If it occurs in the current version of the AI, logs would help.


Do we need to watch out for special characters in this value too? Eg if the tag is `bob-joe`?


Kind of a functional style.
Shall I remove .create() method?


@mckenzielong please, open an issue and if you can post a PR :)


You can shorten this by doing `Invite::firstOrCreate(['username' => $request->username])`


its ok


Untested.


Не знал про contextlib.closing, спасибо.
compresslevel пока не знаю, куда выносить, думаю над этим.


I would swear that I make a comment!


Do you think adding SWING is wise?


TeamCity pull requests :: netty [Build 806](https://garage.netty.io/teamcity/viewLog.html?buildId=3864&buildTypeId=pulls_netty&guest=1) is now running


Change the error message to. Window size cannot be larger than " ". (easier to parse for a user)


Please check other comment and answer to it.


fast-forwarding end-to-end/timeseries to borshop-integration-1295-bugfix/jrd/1293 = 5ae1901f


Тут ты вынес `compresslevel` в параметры, но не используешь его.


Yeah, since this error is specific to the module (some aren't), probably best to leave it unexported.


Hi! Thanks :)
Could you explain your contribution in the PR request description?

Sorry, I'm a bit too purist on coding styles, could you remove the spaces between the colons and the field names?
And squash your commits into a single one?


@branchzero 我这边git clone 不下来，你把我传的 player.swf.png 后缀去掉，把 player.swf 文件传上去吧。


I'd be fine with that, assuming they're not shared across a bunch of files (but I don't think they are now that we have a separate list of Put and Post).


Да, из result нельзя определять потому, что exception может быть валидным результатом (хотя на практике такого сейчас нет).
Я тоже думал про set_exception, наверное, так и сделаю.


Hm, yeah, good point.  I'll have to test and make sure that it won't.


Loaded chunk event freezes game!


Why should this be of numeric type?


>item IDs can change whenever they feel like it in 1.8.8

That *must* be a bug. Please. Please make that a bug. So many mods will
break. Oh my.

2015-12-07 18:36 GMT+01:00 AlexIIL <notifications@github.com>:

> I'm not sure why I don't just send the item ID over the network (Because I
> know its synchronized), and my logic was that it was NBT and I pretty much
> didn't care about that quite as much as to include a full on registry.
>
> What's also annoying (this is in my forum post) is that item IDs can
> change whenever they feel like it in 1.8.8, so I'm going to have to make
> everyone who wants to take an existing 1.7.10 world into a custom version
> of BC for 1.7.10 to transform the id's into the registry ID's.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AlexIIL/BuildCraft/commit/d331cd59ecb867d5d7df04f54bef74ec2fecc6a5#commitcomment-14826963>
> .
>


in my brief experiments i never got this step to work? I havent tried your push. will when i get a better seating arrangement.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8365](https://teamcity.mesosphere.io/viewLog.html?buildId=87706&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


Ignore this file, as the "account" functionality has been removed for the purposes of the class project.


Потому что изменение приватных аттрибутов происходит одновременно и может дергаться из разных потоков. RLock потому, что наследуемые классы при вызове set_result должны тоже атомарно изменять свои члены вместе с членами родительского класса.


Да, в последующих коммитах по способу использования кажется, что `set_exception` будет выглядеть лучше, чем параметр `exception=True`.


nice one... what was i thinking..


TeamCity pull requests :: netty [Build 807](https://garage.netty.io/teamcity/viewLog.html?buildId=3865&buildTypeId=pulls_netty&guest=1) is now running


Assuming this ordering would be much better, we could do exact matching, adding a few more guides to the list (which would be subguides of the current ones there). Make sure we get the important ones. If we miss a guide or two, it would just go to our 404 page, and that's not the end of the world.


Inline images
<img width="1167" alt="screen shot 2015-12-07 at 3 02 56 pm" src="https://cloud.githubusercontent.com/assets/1288192/11636730/44ec46d0-9cfc-11e5-95d3-f2194b0f488e.png">


Wow, that's a nice idea to make everything stateless & neat!


Mind pushing this release to clojars? Thanks ✌


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8366](https://teamcity.mesosphere.io/viewLog.html?buildId=87709&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


the prints are only for debug.. but you could add the same thing to the line below.


Т.е. наследуемые классы будут напрямую использовать `_lock`? Может тогда не стоит его именовать как приватный?


Given we already have something 90% working, that's probably more work than the improvement is worth.


**Code Quality - Critical**

Inconsistencies between function namings. Some functions have space just before the opening parenthesis, while some do not.

Please fix!


Esto es una prueba para automatizar tareas que son comunes con la herramienta de clonación de opengnsys.
El script como tal es funcional sólo queda agregar las rutas reales de los servicios en el servidor donde se aloja opengnsys.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8364](https://teamcity.mesosphere.io/viewLog.html?buildId=87705&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:05:13


Also, I don't know if we ever did make more specific versions of existing articles; if so, it would be a lot of work to find them.


Good call. I'll get that in there. Thanks @sjorge!


doh, meant to grep for those before sending :)


Done - sorry for the delay!


It really seems like changing the version string extraction testing to be unit testing has made adding more cases a whole lot simpler. :+1: :smile:


Why did you change "deliver_now" to "deliver_later" ?


**Code Quality - Critical**

Essentially you are saying if the email has been sent, it should send a response that indicates `Server Error`.

That's the lie of the devil :smile: . Please return a proper json response with the right status code.


Please remove the commented-out code.


Does pyDryad need to be imported here?  It doesn't seem to be used directly in the tests.


Would `(setq truncate-lines t)` be better to avoid the "Truncate long lines enabled" at every popup?


Fixed in #29489.


There should be space between line 122 and 123


this my separate module


Will do


![screen shot 2015-12-07 at 4 09 41 pm](https://cloud.githubusercontent.com/assets/1288192/11636862/0db1255e-9cfd-11e5-90f5-a02b229613aa.png)


Yes it has! :relieved:


`const`


Thx for the fix!


hehehehehehehe


This test isn't necessary and can be deleted.  See the next comment.


I did notice that. That's great, thanks.


random and proportionally sampled


@ifed01 sorry for the nitpicking, but the else should be attached to the closing scope, and have space between the else and the braces


I am so happy.


Why not use `isManaged()` in the other three places?


error figure uploaded


*Correcao, adicionada ao Arquiivo fonte .c


The Ceylon team should really consider using a code formatter.


Arrow syntax!!


**Code Quality - Critical**

The variable name should be more appropriate, something like `$deleteUserStatus`.

Then using a ternary operator, if the user couldn't be successfully deleted, it should return a very nice json response with the right status code.


This should be omitted. The riak node id should just be Ember's `node.id`.
No need for a separate attribute for it.


Isn't `h` after `f`?


Prosím, do prílohy zakomponujte aj zoznam projektov a úpravu projektu.


Reuse the aggregator code? It seems painful to rewrite all the aggregates.


axpeeeer , eti vori filery commit mi tveq eli


pero que bueno bueno
kdhsgdfkj


PR?


this further proves my theory that you're a closeted C developer :wink: i'd stick with `NSParameterAssert` in ObjC since it gives you move information in an ObjC context


There should be space between line 162 and 163


There's a `$` left


Can I ask what this does? Did the syntax change?


It looks mostly good, a few things that remain

- Make sure gl_sarray also has the rolling_apply
- The rolling_apply as of now is not super easy to consume for someone outside. It would be easier to expose this via a function.
- For the built in aggregates, maybe reuse the code from the built in groupby aggregates?


**Code Quality - Critical**

Bad variable name `$e` . Please correct.


out.sql should have the DDLs of all the tables/views related to the query
This is handled by "should contain"

out.sql should not contain unnecessary DDLs
This is handled by "should not contain"

those DDLs should be in correct order
This is handled by "should contain ... before ..."

out.sql contains table stats (UPDATE pg_class ...)
This is handled by ```should contain "WHERE relname = '<tablename>'" ```

out.sql contains column stats if column stats exist before
This is handled by ```should contain "Table: <tablename>, Attribute: <attributename>"```

out.sql can be loaded to create a new database without error
<TBD>


@yifan-gu :  instead of having two channels (events, errs) can we just keep the events channel, and get the error messages from the events?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8363](https://teamcity.mesosphere.io/viewLog.html?buildId=87704&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:18:37
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Hello mick plz get in touch 07943899995


Changes look good.


Not sure what you mean?


> On Dec 7, 2015, at 9:17 AM, atrick <notifications@github.com> wrote:
>
> I don't know if this is overkill or not, but I want to point out a few things.
>
> The scan only happens for unreachable blocks which are typically tiny, so it's hard to say if the extra memory is worth it. This could actually be adding overhead. If hash entries were only added for unreachable blocks, it would be more efficient, regardless of whether it's an analysis.
>
>

I am only hashing entries of unreachable blocks.
> Making this a formal analysis does add some complexity in reasoning about its invalidation.
>
>

I am fine if this is not a formal analysis and I just make a utility + context that I pass in between the loop arc functions.
> As a side note, I want to point out that we should have a clear distinction between a "caching" analysis vs. a "decaying" analysis. A caching analysis such as this would at least never need to be serialized between passes (otherwise making it an analysis would definitely add too much complexity).
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/05ac5962283ca03696c823e959f9ed476501f978#commitcomment-14826468>.
>


Bad Variable name declaration `$e`


I think this assertion may not hold when the loop never runs at least once.


The WB unit tests shouldn't be making live requests against the external providers.  Instead, you should be using aiohttpretty to intercept calls to the expected URLs, and return a mocked response.  So for this test, you'll need to figure out what URL pyDryad will be querying, and call aiohttpretty with that URL as the second argument.  The body should be a hardcoded fixture containing a sample response.   After that, call `provider.metadata()`, then test the return value.  Your tests should pass when you don't have a network connection, because you're mocking your external queries.  The s3 provider tests are a pretty good example of this.  They handle some complex mocking and use the external library (boto) to construct the urls.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8367](https://teamcity.mesosphere.io/viewLog.html?buildId=87715&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


I would recommend, instead of this change, having this line be:

```yaml
run_list:
  - recipe[et_apache_test::default]
```


Please return appropriate json responses like i pointed out earlier on in all the functions


C'était chiant à faire ! J'en profiterai pour faire des tests unitaires en même temps ;-)


Hey sorry for the delayed response. This is indeed an issue, but the right fix should be applied to https://github.com/Asana/asana-api-meta/blob/master/src/templates/ruby/resource.ejs as this is an auto generated file.


@yifan-gu:
can we just remove AddEvent and change AddEvents to AddEvents(events ...*ContainerSyncEvent)?

@Random-Liu:
Yeah, there is really some confusion here.
Because `KillPod()` also returns `PodSyncEvent` now, maybe the functions here should be like this:
```go
func (podEvent *PodSyncEvent) AddContainerEvents(newContainerEvents ...*ContainerSyncEvent) {}
func (podEvent *PodSyncEvent) AddPodEvent(newPodEvent *PodSyncEvent) {}
```

In fact, I'm even not sure whether we need `PodSyncEvent`. Now it is just a collection of `ContainerSyncEvent`, but I don't know whether there will be any "pod level" events we should care about in other runtimes. Maybe we could remove the `PodSyncEvent` and just return a slice of `ContainerSyncEvent` instead to keep it simple.

Because I'm really lack of understanding of other runtimes, I sent this WIP PR. :)

@yifan-gu:
As for rkt, it actually only has pod level events for now, so once there's an error, all the container sync events will share the same error. I think maybe we can also propose something like

```go
type PodSyncEvent struct {
    Name string
    Event pleg.PodLifeCycleEventType
    Error error
    ErrorMessage string
    ContainerEvents []*ContainerSyncEvent
}
```

Which gives pod level runtimes more flexibility?

@Random-Liu:
Good point!

But if we design it like this, the `PodLifeCycleEventType` now may not be enough. We probably should set the notion of event aside, and design it as a well-formatted error reason and message collection of `SyncPod()`.

However, that may need more thinking, because we need a abstraction which works well for all kinds of runtime, at least for docker, rkt and probably hyper now.

For example, a very simple version may be something like this:
``` go
type SyncPodResult struct {
    PodFullName string // Or name, namespace and uid
    Action PodSyncAction // May be Create, Kill, Update?, Else?
    Error error
    ErrorMessage string
    ContainerResult []*ContainerSyncResult
}
type ContainerSyncResult struct {
    Name string
    Action ContainerSyncAction // Maybe Create, Kill, Update?, Else?
    Error error
    ErrorMessage string
}
```

So that we can set both `ContainerState` and `PodPhase` according to the `SyncPodResult`. I found that there are also `Reason` and `Message` in PodStatus, which haven't been fully used. The `SyncPodResult` may also be useful for those fields. This may also be related to #17870.

BTW, in this PR it is wrong that I saved the error of infra container into the `ContainerEvents`, it should be a "pod level" error.


This could be written in a more coffee-script like manner

```coffeescript
angular.module('ponyfm').directive 'colorbox', ->
  restrict: 'AC'
  link: (scope, element, attrs) ->
  	$(element).colorbox attrs.colorbox
```


Is Test::Harness now dead as a project?


I see why you ended up with an empty run_list in the TK config though; still leaning on Vagrant’s config; I forget if, when you have a Vagrantfile, if Test Kitchen uses it…


it's ok. Great!


Will cause a NO_INPUT


Shoot, you're right. That branch hasn't been kept up-to-date. I think the relevant changes are only in the shinydashboard repository, so unfortunately there isn't a really easy way to keep track of them right now.


You've accidentally converted these spaces to tabs. This should be reverted.


What's wrong with mutating this here? It has already created a copy of the data because of `{...this.props.options}`


вообще я предлагал избавиться от обоих (обеих) =)


@jmunhoz, maybe `if (s->aws_auth_needs_complete) ret = -EPERM` or something like it? This won't crash the gateway, and be visible enough


зачем эта обертка? к тому же, span это инлайновый элемент, а не блочный


Not sure how to do that, since this takes a range, which is especially needed if we're going to do median.


I wrote that first too! but doesn't it still run the `ensure`?


One tiny comment from my side:

Please do not remove the primary author of this plugin. Instead, leave it and add your self using a comma like:

<Value author="Alpha, yvesdm3000" />

(...and make the "Thanks" emtpy again.)

Besides from what I am following so far it looks nice! :-)


Isn't that the event loop (i.e. the signature should be `void *_event_loop(void *arg)`? Also line-breaks not according to coding convenients.


Same


> On Dec 7, 2015, at 10:11 AM, Nadav Rotem <notifications@github.com> wrote:
>
> The Key is a pair of pointers (plus other stuff). BlotMapVector does not allow you invalidate parts of the key. So, if you want to invalidate a pointer you need to iterate over all of the keys in the map.
>
Ah ok, I see what you mean. The key part is the invalidate parts of the key. SGTM.
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/92b56cb96d811e43a04c6217b55382f95bbd9ac6#commitcomment-14827798>.
>


Right. Somehow I missed those. Thanks!


This should be boolean isActive() {...}
That way, you can then access the method as a property (e.g.: planDefInstance.active)


I will merge this into thoughtbot/dotfiles as quickly as possible if you suggest this.


This is not equivalent, `before(:all)` is only run once, `let` resets for each example.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8368](https://teamcity.mesosphere.io/viewLog.html?buildId=87717&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


Why not just in the current directory using "./" you might run this on a server automatically without having a Desktop folder


Tim Jenness writes:
> Can you squash this commit with the one that did the initial fix?

Done.


`digest` is called in one of the challenges iirc.


This is the place where the temperature reading is accepted as current (sensorTemperature_); initially as a function of the number of entries in the filter. Now there is one more criteria when determining is the temperature "current" or not; so it is applied here.

Checking the temperature for invalid values is applicable only to the case where the temperature might be accepted as current. Thus somewhere around here.
This is the place where the temperature reading is accepted as current (sensorTemperature_); initially as a function of the number of entries in the filter. Now there is one more criteria when determining is the temperature "current" or not; so it is applied here.

Checking the temperature for invalid values is applicable only to the case where the temperature might be accepted as current. Keeping it local for (somewhat) easier understanding of the code and not as much as an optimization.


good point


@bschonec Thanks for this :+1:
When you add the description to the readme, I'll add the spec tests.

Deal ? :smirk:


Thank for the fix :)


@PBruk or I are going to restore lines 31 and 61-67 and re-deploy to test.


It needs to be for mean and most other aggregators. My vision was to multiplex on the aggregate function name there if something could work on something non-numeric, but currently all aggregators can only take those types.


Where does velocityEventData come from now?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8370](https://teamcity.mesosphere.io/viewLog.html?buildId=87719&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


zukulento


Please let's handle security patches like we always do next time, by keeping it private until the release.


I want to manage colors in my own plugin.
On Dec 7, 2015 9:04 AM, "Flavius12" <notifications@github.com> wrote:

> You can customize colors and chat format simply using this plugin plus
> another one which manages chat format (like CustomChat)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EvolSoft/PerWorldChat/commit/8cc446184eaf5a3bc0f4afd93f3465f72f74458c#commitcomment-14826160>
> .
>


It might seem like a minor optimization, but these add up. 2-3 seconds is already a bit slow for the amount of examples IMHO.


I think this function could be improved a lot by either storing the last element of the list somewhere so that you don't have to traverse the list entirely every time you load an extension.

What I recommend however is to add to the front of the list instead. I don't see how appending to the end provides any benefit here.In lisp you would do something like:

``` lisp
(setq g-extensions (cons new-extension-added g-extensions))
```


These conversion steps when using _expensivePropertyListConversion are really ugly at the moment (hopefully won't be needed in the future), but I needed them to get this to compile and not throw any warnings.


After an Exception is thrown, this tokenizer becomes useless as you don't increase the `count` variable when this happens. Is that acceptable for the client code?


if the ensure is at the method level it will:

```ruby
def test1
  return if true
ensure
  puts 'test 1'
end

def test2
  return if true
  begin
    # no op
  ensure
    puts 'test 2'
  end
end

> test1
test 1
=> nil
> test2
=> nil
```


Same file, it's just defined in a directly usable form.


@pmorie I think the answer is yes. This is a good point, I should add document about this.


I wrapped those in `#if os(OSX)` because I have no idea what they will return and if they will even make sense in Linux contexts. What should I do here?


109626134-VariousDocumentationIssues (mjwitte) - x86_64-MacOS-10.9-clang: OK (1972 of 1972 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


![Go Team!](https://media.giphy.com/media/UatRnEUNX8iCQ/giphy.gif)


I think we should talk about it for a week first.


oh then its not wrong due to mutation, its wrong due to it not responding to prop changes


basically your regex serves two purposes:

1. get the first path element (to extend mod_name)
2. check whether it looks like a collection endpoint

=> you could create a function "get_controller_name(..)" which returns either x_router_controller or the first path element
=> you could create a function "get_function_name(..)" which returns either method.lower() or the collection endpoint name if the path looks like a collection (the logic could be in another function "looks_like_collection(..)"

These are my two cents..


On Mon, Dec 07, 2015 at 10:49:04AM -0800, Daniel Chen wrote:
> corrections from @wking

No need for a separate commit just to credit some editorial feedback
;).  I'd just use:

  $ git commit -a --amend

to push typo fixes and whatnot into your original commit (and then
--force push to update the PR).  You'll still have the history locally
in:

  $ git reflog post/swdcretreat-university_course_roundtable

until it expires if you want to undo a particular change.


:trollface:


It's okay.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8373](https://teamcity.mesosphere.io/viewLog.html?buildId=87725&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:01


In WB v1, folders and folder-like entities should always have a trailing slash, while files should never have a trailing slash.  If I get the metadata for a package, its `path` should look like `"/doi:10.5061/dryad.1850/"`.  The duty of `v1_validate_path` to make sure that folders end with a slash and files don't.


Ah, thanks.


Using instance variable directly was the part that bothered me but maybe I'll just use an accessor.

I'll ping you on the PR


Specifically: https://github.com/flutter/flutter/blob/f162555648c833336f148dfe419a02d577d6e4b0/packages/unit/benchmark/gestures/velocity_tracker_data.dart

For some reason github hides it from the patch?


Better to use very simple building blocks (functions) here, otherwise nobody knows what it does..


done


Updating to canv.3d with Beru changes. Prototypes in canv.3d


@chenji-kael we want to avoid the need to update the clients in order to use the functionality. There is another structure for unsigned entities.


Probably because it's so large.  I misread the patch as removing the file instead of modifying it, hence my confusion.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8374](https://teamcity.mesosphere.io/viewLog.html?buildId=87726&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


При объявлении resources объявляются автоматически роуты: index, new, create, update, edit, show, destroy. На деле используются не все экшены. Надо перечислять через опцию only. Аналогично, с resources :tasks.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8357](https://teamcity.mesosphere.io/viewLog.html?buildId=87692&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:24


This is a good point, adding to the beginning is easier. I purposefully didn't add an end node because I do not like having two variables to denote one list/container


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8371](https://teamcity.mesosphere.io/viewLog.html?buildId=87720&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:53


Yup, that's why I didn't recommend it, plus it would have to be kept up to date in unload as well, would be messier and worse in general.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/24036/testReport/) for this commit**


Dit lijkt niet te werken, event.key is undefined. Waarschijnlijk op te lossen door event.keyCode te gebruiken(48 t/m 57)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8372](https://teamcity.mesosphere.io/viewLog.html?buildId=87724&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:38


what about tests?


Given that we solve the correctness issue by checking the versioning every time we grab a lock, I believe the separate mechanism to track MD version (without using invalidation messages) is easier to debug, understand and maintain. Besides, for such a simple mechanism/requirement we shouldn't require to couple the syscache existence of an object and make the object part of the syscache just to ensure proper tracking. The solution is also lighter weight as at a high level, it processes the invalidation message (it doesn't, but this is for conceptual discussion) at the originator, rather than receiving the message on every backend and parsing and determining that md version has changed. Therefore, IMHO, this separate mechanism is better than retrofitting into existing syscache mechanism.


C'est bien gentil mais y a ptetre plus urgent que de faire ça


true isn't invalid if you #include <stdbool.h>


Committed by accident?


s/, integer>/, 16-bit integer>/. Otherwise one would have to ready detailed description os struct description to get this information.

Also added "(16 bits integer)" next to value in the struct description.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8375](https://teamcity.mesosphere.io/viewLog.html?buildId=87727&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8369](https://teamcity.mesosphere.io/viewLog.html?buildId=87718&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:12:04


That mean code coverage failed us lol. Will look into that.


Sorry for the lack of clarity in my previous comment. You can rename it to clusterId but leave it under Cluster, since it will be used as the general purpose cluster ID from now on. Thank you.


Real life basket cases. Lol :)


Oh! It's an accident, I'll revert it.


Что бы сделать серый фон


This is so that I can modify the pointer. It is so that I can modify either the `g_extensions`, or the `->next` pointers without having to keep track of which one. If I change this to adding to the beginning of the list, this shouldn't be an issue any more.


how long does it take for you?

I get:

```
Finished in 3.78 seconds (files took 1.22 seconds to load)
```


@avivl @eshel PROFIT


Hmm, vermoed dat het aan de browser ligt. Het werkte met Chrome op Linux voor mij, waar heb jij mee getest?


Согласен. Исправлю.


Like that?


it varies, between 2 and 4 seconds.


Ah, yes line 828 is wrong... should be
if equals_0(h_expr): return "(!%s & %s)" % (sigs[0], l_expr)
instead... now that I've got a full truth table, I'll fix it.
If you have a test case for this, I'd be happy to use it.


Chrome op OSX hiero, ook met extern toetsenbord (event.key lijkt ook niet te werken op de jQuery website). Hmmmmm.......


:clap: thanks! (Great release too btw)


Ок. Исправил.


Not without `mao`, because I need to make use of the `NetworkOPs` submission queue.


there's already a case for this, it previously just had no position.


Вынести в locales


done.


Maybe this function should return a truthy value on success? Otherwise return nil as it currently does.


TeamCity pull requests :: netty [Build 806](https://garage.netty.io/teamcity/viewLog.html?buildId=3864&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6222, ignored: 21 Build time: 00:40:28


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8377](https://teamcity.mesosphere.io/viewLog.html?buildId=87729&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:30


done


What's the rationale for such a code cleanup? In my opinion, it doesn't change anything, it only makes it harder to find real changes in the git history. Also the indentation of DESCRIPTION and ARGUMENTS in the doc string is against the convention of how all PyMOL command doc strings are formatted.

This reminds me of the autopep8 branch merging, which I disliked, but missed the chance to reject the pull request. Don't get me wrong, I very much like clean code, but I don't like to make it unnecessarily hard to compare two versions of a script with tons of formatting changes. Consider that many authors of these scripts here will maintain their local copy, and eventually try to update it in the repo.


Action show для отображения ресурсного объекта User, а показываются tasks. Надо сделать хотя бы с привязкой к пользователю


you probably want to call cancelTransaction() ? or is it just a placeholder until we add the 5.5.2 jar?


I assume the centered indentation was intended


should be false


Renaming the drClusterId would cause another annoying side effect, voltdb will refuse to accept the old catalog.jar because it contains name the compiler doesn't recognize. So it's better to leave it there.


Использовать фикстуры или фабрики (factory_girl)


James;
Sounds great, thank. From my perspective I'd love to have more packages supporting OSX but also don't have a local machine to test with so the process to get something running for Macs is pretty painful relative to local debugging (submit a PR, have TravisCI test it, wait 20 minutes, repeat on failure). There aren't any barriers to improving OSX package support -- it would only take interested folks to help with testing, fixing and whitelisting additional recipes. If a high percentage of packages were available for both, changing to a blacklist would make a lot more sense. Thanks much for all the thoughts.


why not just calling the package `c`?


Yes.


text input


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8378](https://teamcity.mesosphere.io/viewLog.html?buildId=87733&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:42


Надо оперировать объектами, а не атрибутами.
task.user == current_user


shouldn't that be ```tlliMalloc``` ?


LGTM. don't forget to change failTransaction --> cancelTransaction later.


tlliMalloc? :P


PyMOL convention is to not indent these headlines, but to indent the following paragraph


That's probably still too high.  Set it to either 0.001 or 0.0001.


dito


You probably wanna check that ```*val``` isn't null after the allocation.


same here


Дублирование реализации логирования пользователя. Раньше в контроллере появлялась уже аналогичная логика. Чтобы в случае изменения логирования пользователя не менять везде код, надо вынести этот код в отдельный модуль.


Ugh, I somehow missed that you introduced the begin, sorry for the noise.


@pjvohra do press ahead - we will get this code sorted out when we talk - unless you do it before that!


@MarcoFalke I took the other changes out per request of laanwj. This commit should not be part of the log at this point.


Для задания отличающихся стилей на внутренних страницах


109626134-VariousDocumentationIssues (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1978 of 1978 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


This block could be refactored as

```
if (err) {
  return err;
}
_setContextMenuItemsVisibility(isPresent, [Commands.FILE_RENAME, Commands.NAVIGATE_SHOW_IN_FILE_TREE, Commands.NAVIGATE_SHOW_IN_OS]);
```


@narfbg this line is hanged my captcha page. If the line (and only this line) is commented all works as expected. I do not understand where the error so leave it to your discretion and did not offer any patch.


because it's more the family of C and derivatives. E.g. indent will work also for derivatives I guess.


clang works for objective C


Not sure I understand the indexOf("[]")?


normalize подключил, но сбрасываю по привычке. Для пущей уверенности :)


Ок. Учту.


Isn't Null<Dynamic> the same as a regular Dynamic? I thought you only meaningfully needed Null<T> when T is some non-nullable basic type like Int or Bool or Float


smart hack


Please squash all of these into the original commit next time.


Да. Это косяк. Постараюсь найти и переделать все подобные места.


109626134-VariousDocumentationIssues (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Maybe mockRequire -> mock to keep it consistent with previous one?


I am seeing interesting behavior with AA caching implemented in 92b56cb96d811e43a04c6217b55382f95bbd9ac6. This is the latest compilation time.  I only see small difference with and without the AA cache in compilation time for DSE.

I performed an experiment last week by implementing a mayReadFromMemory cache in the DSE pass. Below is the compilation time.

Running Time	Self (ms)		Symbol Name
15567.0ms   26.6%	86.0	 	       swift::SILPassManager::runFunctionPasses(llvm::ArrayRef<swift::SILFunctionTransform*>)
2865.0ms    4.9%	10.0	 	        (anonymous namespace)::SimplifyCFGPass::run()
2491.0ms    4.2%	14.0	 	        (anonymous namespace)::ARCSequenceOpts::run()
2361.0ms    4.0%	9.0	 	        (anonymous namespace)::SILCombine::run()
1730.0ms    2.9%	119.0	 	        (anonymous namespace)::RedundantLoadElimination::run()
1684.0ms    2.8%	88.0	 	        (anonymous namespace)::SILCSE::run()
1404.0ms    2.4%	36.0	 	        (anonymous namespace)::DeadStoreElimination::run()
773.0ms    1.3%	136.0	 	        (anonymous namespace)::DCE::run()
440.0ms    0.7%	9.0	 	        (anonymous namespace)::SILCodeMotion::run()
300.0ms    0.5%	6.0	 	        (anonymous namespace)::ABCOpt::run()
213.0ms    0.3%	23.0	 	        (anonymous namespace)::SILSROA::run()
205.0ms    0.3%	2.0	 	        (anonymous namespace)::SILMem2Reg::run()
166.0ms    0.2%	5.0	 	        (anonymous namespace)::ConstantPropagation::run()
151.0ms    0.2%	50.0	 	        (anonymous namespace)::SILLowerAggregate::run()
111.0ms    0.1%	1.0	 	        (anonymous namespace)::StackPromotion::run()
78.0ms    0.1%	46.0	 	        (anonymous namespace)::RemovePinInsts::run()
58.0ms    0.0%	1.0	 	        (anonymous namespace)::LoopRotation::run()

With the AA cache implemented in 92b56cb96d811e43a04c6217b55382f95bbd9ac6.

Running Time	Self (ms)		Symbol Name
15454.0ms   26.7%	85.0	 	       swift::SILPassManager::runFunctionPasses(llvm::ArrayRef<swift::SILFunctionTransform*>)
2771.0ms    4.8%	9.0	 	        (anonymous namespace)::SimplifyCFGPass::run()
2525.0ms    4.3%	15.0	 	        (anonymous namespace)::ARCSequenceOpts::run()
2369.0ms    4.1%	8.0	 	        (anonymous namespace)::SILCombine::run()
1653.0ms    2.8%	146.0	 	        (anonymous namespace)::RedundantLoadElimination::run()
1635.0ms    2.8%	103.0	 	        (anonymous namespace)::SILCSE::run()
1582.0ms    2.7%	42.0	 	        (anonymous namespace)::DeadStoreElimination::run()
771.0ms    1.3%	121.0	 	        (anonymous namespace)::DCE::run()
399.0ms    0.6%	7.0	 	        (anonymous namespace)::SILCodeMotion::run()
279.0ms    0.4%	11.0	 	        (anonymous namespace)::ABCOpt::run()
211.0ms    0.3%	41.0	 	        (anonymous namespace)::SILSROA::run()
203.0ms    0.3%	3.0	 	        (anonymous namespace)::SILMem2Reg::run()
144.0ms    0.2%	1.0	 	        (anonymous namespace)::ConstantPropagation::run()
135.0ms    0.2%	57.0	 	        (anonymous namespace)::SILLowerAggregate::run()
112.0ms    0.1%	0.0	 	        (anonymous namespace)::StackPromotion::run()

Without the AA cache.

Running Time	Self (ms)		Symbol Name
15583.0ms   26.9%	72.0	 	       swift::SILPassManager::runFunctionPasses(llvm::ArrayRef<swift::SILFunctionTransform*>)
2838.0ms    4.9%	7.0	 	        (anonymous namespace)::SimplifyCFGPass::run()
2548.0ms    4.4%	7.0	 	        (anonymous namespace)::ARCSequenceOpts::run()
2324.0ms    4.0%	6.0	 	        (anonymous namespace)::SILCombine::run()
1669.0ms    2.8%	115.0	 	        (anonymous namespace)::RedundantLoadElimination::run()
1661.0ms    2.8%	106.0	 	        (anonymous namespace)::SILCSE::run()
1592.0ms    2.7%	27.0	 	        (anonymous namespace)::DeadStoreElimination::run()
791.0ms    1.3%	129.0	 	        (anonymous namespace)::DCE::run()
381.0ms    0.6%	11.0	 	        (anonymous namespace)::SILCodeMotion::run()
272.0ms    0.4%	7.0	 	        (anonymous namespace)::ABCOpt::run()
215.0ms    0.3%	35.0	 	        (anonymous namespace)::SILSROA::run()
175.0ms    0.3%	0.0	 	        (anonymous namespace)::SILMem2Reg::run()
170.0ms    0.2%	54.0	 	        (anonymous namespace)::SILLowerAggregate::run()
167.0ms    0.2%	1.0	 	        (anonymous namespace)::ConstantPropagation::run()
117.0ms    0.2%	0.0	 	        (anonymous namespace)::StackPromotion::run()
83.0ms    0.1%	55.0	 	        (anonymous namespace)::RemovePinInsts::run()
53.0ms    0.0%	3.0	 	        (anonymous namespace)::LoopRotation::run()
47.0ms    0.0%	45.0	 	        (anonymous namespace)::DeadObjectElimination::run()


Added the comment https://github.com/SeasonFour/Mpolice/commit/5c5d052f4f5635d710b89a865ec35fec5b2fa0aa


We need these improvements in any case


done


this behavior is also implemented as a category: `wmf_mapAndRejectNil:`


ack


So the way I understand this is that you are relying on the init function to register functions in the extension, as well as the cleanup function. I don't consider this very good and reliable, because first off you're only forcing the extension to have an init function but destroy is optional?

You should instead track which functions were registered by an extension and handle the removal of those. You can do this either by setting some recording state that will make tlliAddFunction consider any subsequent calls to be added by said function and therefore fill a list of functions mapped to the extension. Another way is to have the init function return an array of functions to register and then you can simply put that in the map instead of having some kind of stretchy buffer / list.

Either way you really want to avoid the case where an extension adds a bunch of functions and then doesn't remove them. Because then you might have the user calling functions that have been unloaded which will be invalid memory and probably lead to a crash. Which you probably shouldn't get to do from the "safe" environment.


I'm OK with this for temporary purposes but in general I prefer to have buildtools build with a older version of itself to eliminate problems where we depend on a task or tool that we haven't yet built.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8379](https://teamcity.mesosphere.io/viewLog.html?buildId=87750&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


Seems that `CK_NOT_DESIGNATED_INITIALIZER_ATTRIBUTE` would be more appropriate here.


@jackpoz why 3 for iterations when you can just use:

```
for (int i = 0; i < QUEST_EMOTE_COUNT; ++i)
{
    DetailsEmote[i] = 0;
    DetailsEmoteDelay[i] = 0;
    OfferRewardEmote[i] = 0;
    OfferRewardEmoteDelay[i] = 0;
}
```


this reject shouldn't be necessary, as `allLanguages` should never contain `NSNull`, and `bk_select` doesn't inject `NSNull`


If an exception is thrown, the parsing stops immediately, so no need to maintain the tokenizer I think.


@tongzahub the latest available version is `2.7.0`:
`compile 'com.o3dr.android:dronekit-android:2.7.0'`


Cool stuff.


need it for Tab 2


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8380](https://teamcity.mesosphere.io/viewLog.html?buildId=87751&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


This return keyword is unneeded.


The loop *has* to run at least once, because count starts at -1 and position at 0. If the string is empty, character() will never be called, and if it's nonempty, the loop will read the first character from the iterator.


its either NUL char (the ascii null character) or null byte. As far as I know NUL byte (or NUL-byte as seen in buffer.c) is wrong. Isn't it?


can you reduce this to just the stuff you need for testing?


There wasn't an original commit to squash. I just committed them as I finished them.


I should indeed enable "format on save" ;)


I found the problem finally by myself: Somehow `s.stickyElement.outerHeight()` is not working correctly in all cases. I have replaced

```
s.stickyWrapper.css('height', s.stickyElement.outerHeight());
```

with

```
wrapperOuterHeight = $('.stickyElement').css('padding-top') + $('.stickyElement').css('padding-bottom') + $('.stickyElement').css('height');
s.stickyWrapper.css('height', wrapperOuterHeight);
```


Да, действительно. Периодически забываю о возможностях препроцессора.


prefer table-driven testing


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/24040/testReport/) for this commit**


ack


Hello John. I have checked the code and I think you are totally correct about the last 3*(1<<16) threads do not return the correct number. This is because I am concerned about code may use the memory addresses out of what I allocated.


Why is this required to be atomic?


dtto


Why return as numpy array?


Use list comprehension here (one-liner).


How is `num_best` used?


Shouldn't we have a comment on this line mentioning `setup.py` too?


Sure isHttpUri and isHttpsUri are mutually exclusive, but it is still
checking for a valid http or https uri. It won't allow, for example, spaces
in the domain name because that's invalid. It also won't allow
non-http/https schemes like ftp:// or irc://. So the check still does
something, but now it allows https targets along with currently allowed
http targets.

The new option is just a pass-through option to the http-proxy module. It
just specifies if you want to secure the connection through to the target
if you have an https target. The default is true; it'll make sure the
connection to the target host is secure (if it is an https host). But I
don't need the security since the proxy and target are on the same machine
and I can't get node to trust my free StartSSL certificate. Chrome trusts
it but node doesn't and everything I tried to get it to just trust the cert
didn't work.

On Sun, Dec 6, 2015 at 6:19 PM, Manuel Astudillo <notifications@github.com>
wrote:

> are not isHttpUri and isHttpsUri mutually exclusive? seems like the
> exception will never be thrown now.
> Also, why not using isHttpsUri to determine if the target is an https one
> instead of having an extra option?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joelself/redbird/commit/6bb4e34f6ecfe52312912ec92e9f984a3e9d96e5#commitcomment-14811837>
> .
>

--

Thanks,
Joel


Hi @austinXmedic , I often do my loops as you have done here but find that it doesn't take long to get a crap tonne of loops which can become both inefficient and confusing. Perhaps give [this](https://gist.github.com/MosesUK/16ce8bb74dbb57d908fe) a try?


Thanks, I've changed where the 1.1 tag points, but i think ill have to force push to get it to accept the tag moving. I saw something about deleting a remote tag on SO but it iddnt make much sense.

Oh well, its git we should be able to roll back :)


Misleading comment (delete).


How is this `self.index` used?


Yes, please do acknowledge.  If you were a writer/journalist, you'd be at
risk of being called-out for plagiarism, no?

On Sunday, December 6, 2015, Paul Wisdom <notifications@github.com> wrote:

> Hey Chris,
>
> I had you credited at the bottom of my README, but it looks like it may
> have gotten merged out or something. You can see at commit 16f8b14
> <https://github.com/pmwisdom/cordova-background-geolocation-services/commit/16f8b1446f686f33ffa0b87618182b343151a4ef>.
> I'll re-add the credit as your old plugin was initially a starting point
> for android. However how the plugin functions now is completely different.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pmwisdom/cordova-background-geolocation-services/commit/62b29ba2f41fac25077c4957e18c761207767fe0#commitcomment-14811962>
> .
>

--
Snet form Gmail Mobile


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.896](http://teamcity.monogame.net/viewLog.html?buildId=11969&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


We have to free stringType before ABORT_FINALIZE(RS_RET_CONFIG_ERROR);


@pandamicro 如果不重置 那ttf那边的实现可能有问题。。。我再看看吧。


hello! swift!  love you!


collection


break at (


nit: remove one of the newlines (separate top-level stuff by 2 newlines)


In face, looking at the code again, it looks like what you need is a mapping for Harfbuzz’s `hb_buffer_reverse_clusters`. From the docs for: `hb_buffer_reverse_clusters`

> Reverses buffer clusters.  That is, the buffer contents are reversed, then each cluster (consecutive items having the same cluster number) are reversed again.


TeamCity Habitat :: Cruise [Build 330](http://build8ua1:8081/viewLog.html?buildId=811&buildTypeId=Habitat_Cruise) outcome was **FAILURE**
Summary: Tests passed: 97; percentage of statement coverage is too small: 43.456 (new) Build time: 00:01:17


Any reason you left the loop in?


TeamCity Habitat :: Cruise [Build 331](http://build8ua1:8081/viewLog.html?buildId=812&buildTypeId=Habitat_Cruise) outcome was **FAILURE**
Summary: Tests passed: 97; percentage of statement coverage is too small: 43.647 Build time: 00:01:13


Yep, should have been there. It is back.


Tinha trocado o "A" pelo "_", porque estava dando:
Singleton variables: [A]
Porém só depois percebi que com essa mudança o agente não andava mais,
Mas já tinha dado commit.
Desculpa.


Wrong commit. Revert changes im LEMMA 1


Oh wow that wasn't even copied from anywhere haha. Fixing!


idk man


This definitely doesn't seem right. AFAICT status code 210 is the only one which indicates that a room was created and according to all documentation for the "create" method, this is what we should be looking for. Maybe we are trying to workaround a limitation of the library with regard to multiple status codes in a reply? Or are we just changing the semantics of "create" for our own needs?


WFang,

Thank you! Great work! this should put me in the ball park.


You should implement the `meta` backend command instead.


no idea of prior art.

Suggestion: "by contacting one or more project maintainer listed at http://rspec.info/about/"


Is there any thrift library for nim ?


Awesome! :+1: I had the same idea as well but couldn't quite figure it out. Great job!


Not sure, why we added this, need to further check it. I remember it was by the time we added support for joining google multichat ...


I'd be happy with that.


[Sass not SASS](http://sassnotsass.com/)


Wally,

Are you sure about that? It looks to me that we will always return the empty string in that function. Most of the time, start will be smaller than bufSize, since protoSnippet is invoked with a location inside the byte array.
I have experienced an issue that would cause a parser error, and the message would always be "proto=..." until I put back start == bufSize and then I see a snippet of the protocol again.


s/maintainer/maintainers/

I like that.


Suggestion: wrap properties in `code tags` (backticks); e.g. `flex-wrap`


"WIN64=1" is wrong, all the makefile bools take the string "define" or "undef", not 1/0/truthful/falsehoodness. The makefile autodetects win64 based on the bitness of the command prompt. The only time you need to be explicit about WIN64 is when doing a 32 bit build on Win64.


Nit: vendor prefixes for `border-radius` are no longer needed :confetti_ball:


Great article! Just a few nits above.


I think the 2008 part could've probably went into the yearModel variable, but everything else is similar to what I had done, except I had my defaults assigned in Main as `Car myCar = new Car(2012, "Honda");`


Fixes #5


If it's the first one, it seems [taken care of](https://github.com/igniterealtime/Smack/blob/master/smack-extensions/src/main/java/org/jivesoftware/smackx/muc/MultiUserChat.java#L477).
If it's the latter, we can probably switch to using one of the "createOrJoin" methods instead.


ここ段落なんかおかしくないですか？


And when attribute, please provide a link to my original repo, not just
"Christocracy"

On Sunday, December 6, 2015, Paul Wisdom <notifications@github.com> wrote:

> Yep, should have been there. It is back.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pmwisdom/cordova-background-geolocation-services/commit/62b29ba2f41fac25077c4957e18c761207767fe0#commitcomment-14812132>
> .
>

--
Snet form Gmail Mobile


Good work.  An opportunity here would've been to use a Scanner object to allow the user to enter different values for each item, but this still shows good use of objects.


Thank you for investing your time in an attempt to fix this.


Hmmm... I think this I causing an issue...


I agree with the `free()`, but I am wondering about the test, why do we need to perform it before the call?  With the `strcmp()` calls on lines 473, 476, anad 479 all handle `NULL` values without getting an error?  If so, then the check is needed.  But if they don't handle `NULL`, then we need to check for `NULL` right after the `es_str2cstr` in case it fails, right?


Obsolete comment?


It produced incorrect code.


#5 done


Hey Eliot,
  Hope all is well, in this commit I fixed all of your feedback issues up to date :) Please take a look at the question I raised in wiki/_form and also due to integrating epic editor in the form, some feature tests that writes to the 'body' field are broken now. Would be great to work through these issues during our call tomorrow, thank you!!
Best,
Helena


The commit messages is: Jabber protocol now can be invited to gtalk chat rooms and can create such by using the format for the chat room name (private-chat-00000000-0000-0000-0000-000000000000@groupchat.google.com).Fix for clearing connections in every occasion and don't depend on external intervention. Fix chat room ui problem when provider doesn't have any rooms on the server or cannot be queried.


I made lot of changes (profiles, song screen, main screen, etc.). I have a look at my changes and clean up the code first.


@mlutfy that wiki doc you wrote is great. Wish more people would read it instead of just copy-pasting existing code - here's my favorite example of that; the result doesn't even make sense.


Whoops I confused myself by looking at individual commits, sorry.


コメントを振るか関数の名前を変えるなどして、何をする関数なのかをわかるようにしてください（理解力無くてごめんなさい）


The original class did not have the initializer.. I added it so I could assert the unit test.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8271](https://teamcity.mesosphere.io/viewLog.html?buildId=87488&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:09


Removed!


Fixed.


Linux compile error:
`Shared/mods/deathmatch/logic/CLuaShared.h:15:1: error: a storage class can only be specified for objects and functions`


It is because if you use
List<String> weekForecast = Arrays.asList(data);
the resulting weekForecast array will be fixed in size. If you use
List<String> weekForecast = new ArrayList<String>(Arrays.asList(data));
you can still change the size of the weekForecast array after adding the data.

Reference:
http://stackoverflow.com/questions/157944/create-arraylist-from-array


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.897](http://teamcity.monogame.net/viewLog.html?buildId=11973&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Looks like you're leaking the returned handle from this `observeChanges` call. All need to be stopped otherwise the server will keep observing forever.


Could you like the commit itself, please?


good quote :)


Alright, just do it. I'm too tired, night


Psst - write this as `[Maybe.withDefault controlsHtml boardHtml, outcomeHtml]`.


Probably slightly more efficient to do:

    delete handleForProfileNameByIdentityId[identityId];


fixed in open-mpi/ompi@ef03bc726ce6af3abbc4b0bdae603b248cb79627
and backported in ggouaillardet/ompi-release@330b178066daad7f270f9301d7683dbd6792e1a3


Looks like you misspelled canola


There are so many stuff codes. Sorry.


大神  牛逼


Agree.  So it should be like this.
char *stringType = es_str2cstr(pvals[i].val.d.estr, NULL);
if ( NULL == stringType )
{
    errmsg.LogError(0, RS_RET_CONFIG_ERROR,
 						"omczmq: invalid stringType");
    ABORT_FINALIZE(RS_RET_CONFIG_ERROR);
}
Before return, just free() it.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.897](http://teamcity.monogame.net/viewLog.html?buildId=11973&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:07:15


fixed in open-mpi/ompi@bfe8e03d9de505083574fe876b7edc51f1e7b793 and backported in ggouaillardet/ompi-release@263271833ee58a072eabf8444b0bf44e0db2eb4d


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8272](https://teamcity.mesosphere.io/viewLog.html?buildId=87489&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:09


also solve missing -S when passthru is set in config, only in "Basic Linux".


@edgargabriel can you comment on that ?
btw, ```MPI_Address``` is now deprecated and ```MPI_Get_address``` should be used instead


Now you are an owner! \o


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8273](https://teamcity.mesosphere.io/viewLog.html?buildId=87490&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:11


GetIP


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8274](https://teamcity.mesosphere.io/viewLog.html?buildId=87491&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:08


There's already two different ways to get this:

DependNode.isReferenced()
(direct replacement)

DependNode.referenceFile()
(will return None if it's not referenced)


Ок. Уберу это. normalize подключил.


Hooray! :heart_eyes:


LGTM


for consistency with the whole https://github.com/TrinityCore/TrinityCore/blob/1ddf44caf3b0c9af2cc4169b608f1801c2f6f52a/src/server/game/Quests/QuestDef.cpp


Its like to say BEL byte or CR byte.


note that this is an actual response from a server.


Het is een grote hoop aan inconsistencies tussen browsers. jQuery heeft er een prototype voor.


You can use a different service called AppVeyor for Windows testing - http://www.appveyor.com/docs
https://github.com/JuliaLang/Example.jl/blob/master/appveyor.yml
https://www.youtube.com/watch?v=mbG-rqDCNqs


can these identical function definitions be refactored to one spot?


107310700-change-license-text (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1977 of 1977 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@Herschel does that look right?


hello i am tsengel from mongolia
i have a request and i would like to rename mn table
the general name is Mongolian computer braille
and table version name is Mongolian UEB grade one table
best regards
tsengel


Instead of `__FILE__` you should have `_PS_MODULE_DIR_`. Same thing for line 467, 548, 553, 682, 687.


@Uaman було лажово, ти виправив на таке саме, зробив би вже по-нормальному


Да. Сделаю модификатор.


@Uaman може зробиш по-нормальному?


> [Eau Claire](https://github.com/whackashoe) replaced a function that was marked as unsafe by Visual Studio.

Think you copied wrong bit - username is whackashoe


See PR 612 and 613 for test case and suggested fix.


Честно говоря, сам не знаю. Подумал, что так будет лучше.


I'm not longer aiming towards that goal, so I've deleted the link. I'm planning on discontinuing this plugin as a First Data solution as it can't really be re-factored for the new API and I just don't have the time anymore :(


RIP macroz


Я пока не пробовал использовать примеси. Но видимо уже пора.


@Simn I wonder if the feature name should be simply `haxe.Int32.mul` instead.


Учту.


I'm going to ask cpw about it, just to check if we should be using ID's at all any more though


also unused?


Thanks! I forgot about the C++14 feature, obviously. Will fix, ASAP.


errado


`getStackTrace` would return a Java array of `StackTraceElement`s, which would be none too pretty.  This should probably use `ExceptionUtils.getStackTrace`


Yo, I think this may have corrupt the Droid Sans font files. Did you do a search/replace, or did line endings get converted on you?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8382](https://teamcity.mesosphere.io/viewLog.html?buildId=87753&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


The response from Box itself doesn't give an `expires_at`, only an `expires_in`, and it was previously being calculated before it was saved into the `ExternalAccount` object. Switching this kwarg to accept a function for extracting the correct datetime representation of `expires_at` instead, with a default being provided by `ExternalProvider`.


I'm not sure if I understand the issue. I'm defining the callbacks to be used, and in this test we use a token string and a TokenDetails.
This other test https://goo.gl/ckt6Rn uses a TokenRequest.


Переделал. Псевдоэлементы нельзя использовать в JS


Sorry, what? I used "optimize imports", which changed the order of imports, if that's what you're talking about?


Doesn't this do the wrong math?


Oh, sorry, I didn’t realize that was the test file. I thought you’d added some tests in the main module for local testing and then accidentally added this :D


the é character throws an error during manual/make.  Ren\'e fixes it.


Noting that this broke integration with the [cf-uaa-lib](https://github.com/cloudfoundry/cf-uaa-lib) gem.


I didn't see that issue--can you please try a fresh build?


haha no :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8381](https://teamcity.mesosphere.io/viewLog.html?buildId=87752&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:58


I digged in a bit further and the differences come from in my last week experiement i am caching memorybehavior, which does more things than just querying AA. I talked to Erik and we think we need to follow the same philosophy as the AA cache and introduce a MemoryBehavior cache, especially when Erik's escape analysis (which is called in MemoryBehavior) becomes more expensive than what it is now

This is without the memorybehavior cache (but with the AA cache) in DSE.

Running Time	Self (ms)		Symbol Name
642.0ms    1.0%	17.0	 	           swift::AliasAnalysis::computeMemoryBehavior(swift::SILInstruction*, swift::SILValue, swift::RetainObserveKind)
624.0ms    1.0%	35.0	 	            swift::SILVisitor<(anonymous namespace)::MemoryBehaviorVisitor, swift::SILInstruction::MemoryBehavior>::visit(swift::ValueBase*)
338.0ms    0.5%	6.0	 	             swift::isNonEscapingLocalObject(swift::SILValue)
126.0ms    0.2%	16.0	 	             swift::SideEffectAnalysis::getEffectsOfApply(swift::SideEffectAnalysis::FunctionEffects&, swift::FullApplySite, bool)
40.0ms    0.0%	6.0	 	             swift::getUnderlyingObject(swift::SILValue)
32.0ms    0.0%	4.0	 	             swift::AliasAnalysis::alias(swift::SILValue, swift::SILValue, swift::SILType, swift::SILType)
23.0ms    0.0%	0.0	 	             swift::SILInstruction::getMemoryBehavior() const
17.0ms    0.0%	0.0	 	             swift::SILInstruction::mayHaveSideEffects() const
7.0ms    0.0%	1.0	 	             swift::computeTBAAType(swift::SILValue)

This is with the memorybehavior cache (with the AA cache) in DSE.

69.0ms    0.2% 5.0                        swift::AliasAnalysis::computeMemoryBehavior(swift::SILInstruction*, swift::SILValue, swift::RetainObserveKind)
162.0ms    0.2% 162.0                      bool llvm::DenseMapBase<llvm::DenseMap<std::__1::pair<swift::SILInstruction*, swift::SILValue>, bool, llvm::DenseMapInfo<std::__1::pair<swift::SILInstruction*, swift::SILValue> >, llvm::detail::DenseMapPair<std::__1::pair<swift::SILInstruction*, swift::SILValue>, bool> >, std::__1::pair<swift::SILInstruction*, swift::SI
LValue>, bool, llvm::DenseMapInfo<std::__1::pair<swift::SILInstruction*, swift::SILValue> >, llvm::detail::DenseMapPair<std::__1::pair<swift::SILInstruction*, swift::SILValue>, bool> >::LookupBucketFor<std::__1::pair<swift::SILInstruction*, swift::SILValue> >(std::__1::pair<swift::SILInstruction*, swift::SILValue> const&, llvm::detail::DenseMapPair<std::__1::pai
r<swift::SILInstruction*, swift::SILValue>, bool> const*&) const
40.0ms    0.0%  4.0                        llvm::DenseMapBase<llvm::DenseMap<std::__1::pair<swift::SILInstruction*, swift::SILValue>, bool, llvm::DenseMapInfo<std::__1::pair<swift::SILInstruction*, swift::SILValue> >, llvm::detail::DenseMapPair<std::__1::pair<swift::SILInstruction*, swift::SILValue>, bool> >, std::__1::pair<swift::SILInstruction*, swift::SILValu
e>, bool, llvm::DenseMapInfo<std::__1::pair<swift::SILInstruction*, swift::SILValue> >, llvm::detail::DenseMapPair<std::__1::pair<swift::SILInstruction*, swift::SILValue>, bool> >::InsertIntoBucketImpl(std::__1::pair<swift::SILInstruction*, swift::SILValue> const&, llvm::detail::DenseMapPair<std::__1::pair<swift::SILInstruction*, swift::SILValue>, bool>*)
2.0ms    0.0%   2.0                        (anonymous namespace)::DSEContext::getBBLocState(swift::SILBasicBlock*)


Perhaps, that should be a straightforward fix in common.ml's `has_feature`.


Hi, the features are updated


Style issue:  please wrap at 100 columns.


Not that I know what I'm doing with GitHub, but I would approve this pull request - it matches the string names as exported from Foundation on Mac / iOS. @parkera, this is fine to take.


Style issue: please wrap at 100 columns.


Thanks so much for the feedback, @ookla and @johankool!


dbrajkovic/swift-corelibs-foundation seems to be offering the same change, along with some others.  So maybe that should be taken instead.


ugh.. ok.. thanks. I'm in hell trying to deal with firstdata for a client.
Good luck out there.

~~~
Cathy Zollo
941-586-1679 -- Phone
941-866-2665 -- Fax

IMPORTANT: This e-mail is subject to the Electronics Communications Privacy
Act, 18 U.S.C. §§2510-2521, and contains information which is or may be
confidential and/or privileged. The information contained in this e-mail
message, together with any attachments or links contained herein, is
strictly confidential and intended only for the use of the recipient named
above. If the reader of this e-mail is not the intended recipient, you are
notified that any use, distribution, or copying of this communication is
strictly prohibited.

On Mon, Dec 7, 2015 at 3:06 PM, Aubrey Portwood <notifications@github.com>
wrote:

> I'm not longer aiming towards that goal, so I've deleted the link. I'm
> planning on discontinuing this plugin as a First Data solution as it can't
> really be re-factored for the new API and I just don't have the time
> anymore :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aubreypwd/gravityforms-firstdata/commit/e3e64567fda0436fd14e965a104dfbe6cb70887e#commitcomment-14830475>
> .
>


Thank you!


Board has some params e.g name ?
Board.new(board_params)

private
def board_params
    params.require(:board).permit(:name)
end


yeah, it was an c & p error. shouldn't be any return type there


@nrstott, review please.


The app was being seeded with so many expired goals that `rake todo:delete_goals` eliminated more than 75% of the total goals. Now it should only be around 30%.


is there a tab in there?


Never mind I got confused I think it is right.


was a rushed commit I missed it ...


I liked the ``pagesize + size + pagesize`` way because it shows you the guard pages graphically.


Yeah, I really wish I could afford the time to totally re-build this, sorry
for the inconvenience.


That's a pretty big buffer, maybe make it static?


if you using form_for you should do something like this
f.label instead label_tag
prefix tag we are using for 'form_tag'


This changed from ``>`` to ``>=``, is this correct?


sessionOnly should be based on an expiresData of nil, or the value of the NSHTTPCookieDiscard property.


This change conflicts a lot with `develop` branch with merge #84 . Are you working any further at `partial` branch, @shuchenku ? If not, I'm going to try to resolve those conflicts.


This fix will be applied to the school server in about 1hr


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8384](https://teamcity.mesosphere.io/viewLog.html?buildId=87757&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


Backticks are used when you want to capture the output, otherwise `system` is more concise. Kind of similar to `map` vs `each`: `map` iterates *and builds a collection*, if you are not interested in the collection then you use `each`.


The new design is to match the designs of [project manager](https://github.com/danielbrodin/atom-project-manager) of atom and the default sublime "Quick Switch Project". You could still access the options via command palette.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8385](https://teamcity.mesosphere.io/viewLog.html?buildId=87758&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


Зачем удалили? :O


>firmware: Add option to disable touchscreen on Pi display

How can the touchscreen be disabled?


The name of the variables have to be descriptive


Пока только теми, которые описаны в Gruntfile.js - watch (less, postcss)


Ok, thanks for clarifying @hmln, I was not aware of test `test_token_request_can_be_used_to_get_a_token`.  This is all good, :+1:


:+1:, but please pass `provider.NAME` instead.  Otherwise you get error messages like this:

![screen shot 2015-12-07 at 3 23 57 pm](https://cloud.githubusercontent.com/assets/86835/11638762/98f8d42e-9cf6-11e5-8030-e0308f438c72.png)


I think `VersionShown` only works for the method call to show the version (different than the help).  I think maybe we can just do this `methodCalls` stuff directly in `FakeCommandLine`, then replace `VersionShown` by that, how does that sound?  Then we can use it elsewhere as well.


Yes, but now it just takw way more steps to open a project in a new window. :( Could you please add a option for this?


Thanks


This is my first comment


Looks ok nm, just inverted the if/else and condition.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8383](https://teamcity.mesosphere.io/viewLog.html?buildId=87756&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:39


Oh, so that's what that "C" means. Heh... whoops. Thanks for the fix.


Yes, I know. I was just expressing my hopes in even more positive outcome from the changes. :)


Review from this part I don't know why is this part here


I try to add a new file to the repository.


Should still modify the single child if the last parent only has one child


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 826](https://teamcity.labkey.org/viewLog.html?buildId=289994&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 15 (2 new), passed: 247 Build time: 01:32:56
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

##### there are 5 more failed tests, see build details
```


My favorite color is blue.


TeamCity pull requests :: netty [Build 807](https://garage.netty.io/teamcity/viewLog.html?buildId=3865&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6230, ignored: 21 Build time: 01:24:21


And #836


Whoops, also bundled in a change to eliminate some of the less useful optimizations on Pypy, as Pypy does not benefit from local binding.


@alketh I think your version of `roxygen2` is out of date, could you please run:
`install.packages("roxygen2")`
if
`packageVersion("roxygen2")' does not equal ‘5.0.1’


I used [this](https://github.com/randy3k/Project-Manager#optional-keybind) and I think it is faster.


Why we catch all exceptions?


@arjuks When I try to start this intent it isn't working for me. I don't see this actually implemented anywhere in the code. Are you sure this should be an activity? I thought maybe a fragment


How are you invoke to the method waitUntilPageObjectIsLoaded? I think you forgot the constructor with the page factory


109626134-VariousDocumentationIssues (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (904 of 904 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-25.57%25-green.svg)


LGTM


Great job on more the descriptive naming :100:. Generally though, stick to either underscore or camelCase.


There are already two calls to `authorise` on this test. On the first one we set the params. On the second one, we mock the `request_token` method to check if this second call to `authorise` will pass the same params of the first one. Force is used just to ensure that `authorise` will try to issue another token.


Do we know which commit broke the mac binary?


OK. Thats true, :) Thanks.


What about referencing using 1.0.* for version number?

"Microsoft.Xaml.Behaviors.Uwp.Managed": "1.0.*"


@AnriShitko


A few comments:

1) We need to be careful to avoid bash specific syntax, these scripts need to work in `dash`, `ash`, `sh`, etc.
2) The pattern needs to match any number of digits for the partition number,
3) It's not clear that stripping the partition information is the correct thing to do for all configurations.  Arguably we should update the utilities to accept the vdev guid to prevent any confusion.


109626134-VariousDocumentationIssues (mjwitte) - x86_64-MacOS-10.9-clang: OK (1972 of 1972 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


If I recall, there's a HelpShown too


Merging my changes to index.html.


there was a note about it already


Ah, that might be easier in the interim if it exists.


```c
strcpy(use->receiver, who);
```
or
```c
strncpy(use->receiver, who, len + 1);
```
plx


The only reason this could throw is passing invalid url (malformed or non-http) and since we rethrow all exceptions catching them shouldn't be bad (they won't be silent anyway)


Or the other way to do it is "super+n", then "super+ctrl+p".


ok, I've copied the same note of encode_params.


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 827](https://teamcity.labkey.org/viewLog.html?buildId=289996&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 24 (2 new), passed: 256 Build time: 01:34:55
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

##### there are 14 more failed tests, see build details
```


Sorry, I know this isn't properly documented in the base provider, but the signature of `delete()` is `delete(self, path, **kwargs)`.  `upload` has the same issue, it should be `upload(self, stream, path, **kwargs)`


I give it a try later at work. Thanks you.

Randy Lai <notifications@github.com> schrieb am Mo., 7. Dez. 2015 um
21:34 Uhr:

> Or the other way to do it is "super+n", then "super+ctrl+p".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/randy3k/Project-Manager/commit/f3ed7ce36857f6a1eab86aa88291ad0fdde8047c#commitcomment-14831087>
> .
>


Thanks for clarifying @hmln, I misread what the mock does, sorry not that familiar with Python mocks but understand now.


I haven't exactly see a ton of TS code, but from what I have seen, spaces are common around the colon starting type annotation.


disable_touchscreen=1
this is the Pi's DSI LCD touchscreen)


No need to default `path` here.


So I used to be able to move my compilation window to the right with `SPC w L`. Now I actually have to use my mouse to even unfocus the compilation buffer...

I bisected the issue to this commit, and it's still not fixed in `develop`, any ideas @justbur @TheBB @syl20bnr ?


Oh, I missed the byte word. Okay, you win.


Please move all constant to constant class.


style nit: consistent newlines before doc comments.


nit: a->A


no, actually I've changed it a bit to make sense


I think we should choose between camelCase or underscore for the variables naming. I have a preference for the camelCase (moreover, most of the variables from mapael are written in camaelCase for now). What do you think about it ?


I assume you've updated to the latest packages and commits on the develop branch?


TeamCity LabKey :: Sprint :: Premium :: Git postgres 9.4 [Build 3](https://teamcity.labkey.org/viewLog.html?buildId=289986&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitPostgres94) outcome was **FAILURE**
Summary: Tests failed: 24 (3 new), passed: 256 Build time: 01:43:15
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitFooter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testCounts: <no details avaliable>

##### there are 14 more failed tests, see build details
```


nit: otherwise->Otherwise? (or should it not be a sentence, but a phrase? I'm not well with grammar)


This is brilliant.  We are on our way to Caldecott.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8386](https://teamcity.mesosphere.io/viewLog.html?buildId=87762&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


There are already two calls to `authorise` on this test. On the first one we set the params. On the second one, we mock the `request_token` method to check if this second call to `authorise` will pass the same params of the first one. Do you think checking what is passed to `request_token` isn't enough?


looks like it worked after updating packages, derp. thanks


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8376](https://teamcity.mesosphere.io/viewLog.html?buildId=87728&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:10


This doesn't work yet.


Take out the -- and the problem is fixed for when entering two incorrect answers and then the third as correct but still get response saying: incorrect.  - Mike---


Dobry Commit


Rzeczywiscie Bardzo dobry


Minor but we should start following the guidelines at https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/project-guidelines.md. Which means this would become a "net45.cs" file.


Niezle wyszedl


Wzorcowo


Ta jest najlepsza


Ta jest najlepszaTa jest najlepsza


Ta jest najlepszaTa jest najlepszaTa jest najlepsza


Ta jest najlepszaTa jest najlepszaTa jest najlepszaTa jest najlepsza


This line can now be removed.


Ten mi sie nie podoba


Or the other way to do it is "super+shift+n", then "super+ctrl+p".


Mi tez nie


complete


Excellent! Thanks Mark.


@manavgabhawala This test case fails to compile properly.

error: no check strings found with prefix 'CHECK:'
generic_structs.swift:22:10: error: cannot specialize non-generic type 'Bar'
  var b: Bar<A1, A2>

Also, mentioned by FileCheck, you don't have any CHECK: lines. Since you're fixing a crash it's best to just remove the pipe and FileCheck.


complete.


Remove the `(see bower.json)`


Thanks for the commit on this - let me test it out, and then I can merge it.


Where can I find discussion about this change? Why it confuses people and what we should use instead of this "install" target?

We use 'make install' for Linux Arch package. It is a great idea to have system-wide installed library for cross-compiled projects so other projects can easily use it. I haven't seen anything confusion about it.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8390](https://teamcity.mesosphere.io/viewLog.html?buildId=87767&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to start build; error while applying patch Build time: 00:00:00


I am confused as to what the values of b in the constructor and r/d in getHash are and how they relate to the output size. If i wanted an n bit output size, how would i calculate b, r and d?


catch the RuntimeError here


This is now back to 6000 for me. Maybe it varies?


Thanks for the clarification.


A type alias for this function type would make this easier to read.


this is the right way to ensure all outstanding acks have completed for OFI; I just tested this on top of my error-checking OFI patch


Oh, sorry about that. I committed a fix.


Large Kepa should be Kepa Grande, it sounds better.


$1 holds a count of the number of versions of the package that are installed


Can we make these sleep times configurable?  Perhaps call them: PTERO_SHELL_COMMAND_POLLING_INTERVAL and PTERO_SHELL_COMMAND_TERMINATE_INTERVAL.


Передаю в input `*.svg`, оно создаёт папку `*.svg`.


double bang removed
removed extra comments in Gemfile


@Zoschi
could you check this out and let me know what you think?

any ideas why the ground might not be spawning?


spaces before colons


complete


All Rights?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8392](https://teamcity.mesosphere.io/viewLog.html?buildId=87769&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


Agreed, should be 1.0.*


maybe type this as `any` instead of `Object`?


For some reason I was not able to view the third test unless I open up the 'failed tests' section of xcode on my system. I altered my code to pass all tests in this version. Learn is still unable to perceive when I run a local test.


I just opened a pull request for the latest release in Homebrew.

Updating the formula is as easy as updating the url and sha256 for the new release, then opening a pull request to Homebrew. In this case, I also had to update the tests to reflect the new command line syntax.

I can try to watch for new releases and keep this up to date.


Well, that's embarassing :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8391](https://teamcity.mesosphere.io/viewLog.html?buildId=87768&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:56


I reverted the pull request in:

https://github.com/apple/swift/commit/a0f11ca7b9a103baa25bcedd61953405a9f3cd94


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8393](https://teamcity.mesosphere.io/viewLog.html?buildId=87770&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


get this garbage out of here


I'm about 95% sure this should be `payload[key] = self.instance_variable_get(:"@#{key}")`, right? I'm trying to write an adapter for the `SerializationAssertions` module that will work with RSpec and I'd like to be able to assert not just the serializer that was used, but also the object that was serialized.


how do we "view" the logs now that they are compressed? I attempted various gunzip command syntax but always had the error: "not in gzip format"


Zmiana nazw etykiet w menu_add


Thanks for notifying me. That's the same as
https://josm.openstreetmap.de/ticket/12158, right?

I've published a fix as [version
1.1.1](https://github.com/floscher/josm-mapillary-plugin/releases/tag/v1.1.1)
(revision 31802), which should resolve the issue. In the future I'll be
more careful before releasing a new version.

Unfortunately the issue was caused by [the part of MapillaryLayer.init()
that the unit tests
skip](https://github.com/floscher/josm-mapillary-plugin/blob/14e20dca3fbbec3559862932420b8882d3a6c027/src/org/openstreetmap/josm/plugins/mapillary/MapillaryLayer.java#L109),
because that part needs an initialized JOSM-MapView. I'm working on
providing a test case testing for this bug, so that this certain bug
won't reappear in the future.


I get it when running the admin tool with ipython:
```
 PYTHONLIB=. ipython -i --pdb pkgdb2client/admin.py  process
```


complete


I'll open a new issue and submit a PR


If you need help creating your first instance, please reference our getting started guides:


Whats deleted


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8389](https://teamcity.mesosphere.io/viewLog.html?buildId=87766&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:10:49


You're right.  Sorry about that.  I have corrected it.


Better only 1 constructor in the builder, itemType is optional and set to default. So always can be setted in the builder


InternalCommunicator jako argument w metodach WebApiCommunicator


We should commit or rollback (probably rollback) before going back to sleep so that we don't leave the transaction open for the lifetime of the job.


Poprawienie tego (trochę lamerskiego) sprawdzania, czy UUID się udał.
Ogólnie moim zdaniem metoda jest do poprawy (obsługa wyjątku).


@aforward @TimLethbridge I wasn't sure what version of Ruby to install, so I used 2.1. Is that adequate? We could fairly easily add more versions of Ruby to test against, too.


My intent in removing images was to start a conversation about which images are worth including. For example, screenshots of a single button in an uncluttered GUI seem less helpful than a cap with populated fields.


@valib Sorry mate i'm moving and haven't had time to test more to be sure. I will be able to in approx a week


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 89](https://teamcity.labkey.org/viewLog.html?buildId=290023&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:06:53


I removed the step about sanitize and just stated it in a note.


Is it worth adding a callout and extra verbiage?


This removed testing `openjdk` in favour of builds starting and finishing quicker on Travis. I don't think it was really necessary. Later, it'll be easier.


Je devrais mettre ma couleur préféré.


Will take in consideration.


@brentvatne Thank you so much for all your hard work today in resolving my issues! My project is back on track and better than ever, you rock! (Fixed android linear-gradient, video, and mapbox :+1: )


s/should not/should/


109626134-VariousDocumentationIssues (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1978 of 1978 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


That is not so bad


Would be better if the constructor with builder is private/protected? I guess next version will be the only constructor so should be build always using the builder.


I ended up making it not fall through. More readable that way, anyway.


Uma nova "abordagem" que elegante...


complete


У меня аппарат с hdpi разрешением, некоторая графика отображается коряво - так я это исправляю. На аппараты Xiaomi это не влияет же, у всех разрешение выше


I agree that merging lines 16 and 18 works, but the suggested language is cumbersome.


@AkamaiDASH I tried to add the lines you suggested to `updateBufferLevel()` in `Buffercontroller.js`, and the `bufferTarget` value goes down, but I cannot see any noticeable difference. Is there some other attribute that can be changed to reduce the buffer level?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 8396](https://teamcity.mesosphere.io/viewLog.html?buildId=87773&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:23


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1103.4c4d35e27fa4fb51903eb252163a82cd59306fc6](http://build.palaso.org/viewLog.html?buildId=66959&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 742, ignored: 6 Build time: 00:06:05


Try zcat


Done, with apologies for the delay!


No problem, glad to see it fixed this quickly! :)


Why are `image` and `imageSpec` both populated when the  *design* declares them as mutual exclusive?


I tried it on an example project and it works without the `exclude` but I am not sure if thats correct.
GWT documentation doesn't explicitly say that folders marked as super-source are automatically excluded from source path. Thats why I usually add exclude rules in these scenarios. Maybe @tbroyer knows more about it.


:+1:  winou


After some testing it seems that `app/console cache:clear` breaks this.


If you're going to change these to `<encrypted>`, do you want to change them in the CreateUserForm on lines 94-96?


Ball of Duty Server/Domain/Physics/Collision/ICollidable.cs

rename to IsCollidingWith?


oops accidentally removed the links to checklists. I meant instead to replace the thumbnails block with my new partial template.


make sure to actually test this (see https://github.com/Turbo87/intellij-emberjs/issues/4 for example :confused: )


ну как то так получилось, в мэйне очено подробно все проверки продемонстрировал просто, ну и плюс три почти идентичных класса!!!


whoops... sorry about that. broke during the rebase...


This shouldn't happen. If you see these files in that location something is wrong with the buildsystem. The cmake generated files should end up in `project/cmake/addons/build/`.


Sorry, I meant to separate this into a separate commit as well. Yes, I will change in both places.


There are more Mongolian typefaces can be used for test, you can get these typefaces from following sources included in a CSS file:

http://trans.mglip.com/css/css.css


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.907](http://teamcity.monogame.net/viewLog.html?buildId=12013&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


ok bro...
we are providing:
-cam overlay methods with libui
-pmem methods with libbinder
-vectorimpl symbols with libcutils
-some legacy cam symbols with libmedia

here is the list of libs that need to be edited:
1-libseccameraadaptor (4 strings should be edited)  ***
2-libcamera (3 strings) ***
3-libmmipl (1 string)
4-libmmjpeg (1 string)
5-liboemcamera (1 string)
6-libsecjpegarcsoft (1 string)
7-libsecjpegboard (1 string)
8-libsecjpeginterface (1 string)

ı dont know how to edit libseccameraadaptor and libcamera because more than 1 string should be edited and then we must inject "libshims.so" string but if ı delete one line/string..it corrupts lib itself(sht table size corruption) so ım stuck

but other libs that 1 string need to be edited is easy...just replace string with "libshims.so" and no sht table size error....


hmm, I actually never had to do this. it just worked out of the box.


Will it actually roll back if you catch the exception?


because for backward compatibility, we if we don't keep `Image` in the result, programs using that field will have error. https://github.com/kubernetes/kubernetes/blob/ec1ba7438fb2c325a75b8885c7b5ee1b736dfc3d/docs/devel/api_changes.md#on-compatibility


@steveeJ ^


I fetched the previous revision of DroidSans.ttf and DroidSans-Bold.ttf, and they work fine. I'm downloading these through the github UI in my browser.

I did a quick diff on the DroidSans.ttf versions, and it does indeed look like git changed "\r\n" to "\n" on you. If you're looking at these checked out through git, it may have changed them back on you and they'll look fine on your system.


This commit doesn't merge into develop, what's suppoosed to happen?


fixed #10


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/24043/testReport/) for this commit**


Yep, I've tested to make sure.

Any unhandled exception inside `with transaction.atomic()` will rollback the changes.


:+1: Mieux!

Pour les forms, vous pouvez faire ainsi: `name="matchID[{{ $match }}][away]"` pour PHP.


:tada:


Ah perfect.


Just for the record, is there a reason why you use the "_" postfix char on some parameters ? We're not using them so far, unless on some very rare occasion. I for one use the syntax highlight to differentiate class fields and local variables.


This is part of CMake yes? I'm wondering what modifications were made... it would be nice to document them


Do we check if its colliding with wall for every character in game? Thats a bit silly isnt it?


Por las dudas: On the comentario que hiciste de las conclusiones del Capítulo 3, es lo que estaba de conclusiones de MECOM:

```No shear bands are observed, which is to be expected given that our glass was generated with very high quenching rates. Since no shear bands are observed in our simulations, the identification of plasticity is complex. Surely there are shear areas, "shear transformation zones" (STZ), composed of a few atoms that experience high shear stresses. The identification of these areas requires a very detailed observation of the sample, involving much longer simulations than those used here.```


lol, no reason not to rename it, I suppose. :)


109626134-VariousDocumentationIssues (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1978 of 1978 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


ceci ressemble à une méthode statique.


Ball of Duty Server/Domain/Physics/Physics.cs

do we check if its colliding with the wall for every character? Thats a bit silly isnt it? We only want to check if its colliding with the wall once.


@ednawig It would be better to return false if none was found.


Issue committing from Cloud9. Global user.name and user.email appears to be set. Code was actually committed by JFGreene83


@smalenfant Remind me what "dummy" (lowercase) is for?


test


That's my understanding from reading the django docs. It looks if the block has been exited normally or via exception. This should cause a rollback to happen.

Looks fine to me Simon.


Should be deleted


`%q` formats it as a legal Go string, so it's surrounded by quotation marks. (Only metric names and tags don't use quotes).

If you take a string that the language accepts, and encode it in the way that Go supports, then it will still be accepted by the language (so this transformation is "good enough" to escape the string).


test @uyvunk


Can you change to ^3.0?


ouille, ouille. évitez `window.location`.

Testez ceci: `<a onclick="return confirm('êtes-vous sûr?')" href="/test">test</a>`