Sorry, your code blows up when I try to run it on the files generated by the `generate_testcases.sh` script:

    $ env RUST_BACKTRACE=1 ./target/debug/exa testcases --long --tree
    thread '<main>' panicked at 'index out of bounds: the len is 0 but the index is 0', ../src/libcollections/vec.rs:1116
    stack backtrace:
       1:        0x10d17bf58 - sys::backtrace::tracing::imp::write::ha1fecc817f4697fas9s
       2:        0x10d17d8ff - panicking::log_panic::_<closure>::closure.39708
       3:        0x10d17d3a2 - panicking::log_panic::h631dc367e76559d1xYw
       4:        0x10d170ad6 - sys_common::unwind::begin_unwind_inner::h7efd836278b7cb4d0cs
       5:        0x10d170ebe - sys_common::unwind::begin_unwind_fmt::hc18453f57d01abac6bs
       6:        0x10d17b5e7 - rust_begin_unwind
       7:        0x10d19eb30 - panicking::panic_fmt::h6c78ce0128588a957HK
       8:        0x10d19b2a0 - panicking::panic_bounds_check::he3d825b7425f1f15dHK
       9:        0x10d06b6a8 - vec::_<impl>::index_mut::index_mut::h10940914251765809645
      10:        0x10d0686b3 - output::details::_<impl>::print_table::h2120631830446409840
      11:        0x10d03b0f2 - output::details::_<impl>::view::hf18e487ba7e765fbgsd
      12:        0x10d076c6a - _<impl>::print_files::h90e22310646e25bcate
      13:        0x10d075c69 - _<impl>::run::h44554560c55db755lle
      14:        0x10d079205 - main::h1e76b862f41d2fecUte
      15:        0x10d17d122 - sys_common::unwind::try::try_fn::h9087674402355109000
      16:        0x10d17b428 - __rust_try
      17:        0x10d17cfc9 - rt::lang_start::h424562221c19d8b9FVw
      18:        0x10d07a959 - main

Changing it back to `resize` instead of `reserve` fixes it. Turns out, the row depth *can* decrease (such as in [this case](https://github.com/skade/exa/blob/77fa8974c4cf3117c1c83a5ab41e059336b5a42c/src/output/details.rs#L81)).


Woops, actually regressed in 5a1d4e5ff987d3aeb0549a29004d0a71fc34ba91


Yes, the spec says:

> 2d. If any of the following are true, then classify as a conflict:
> * there are pending notifications of changes to foo;
> * the last-uploaded statinfo is either absent, or different from the current statinfo;
> * either last_downloaded_uri or last_uploaded_uri (or both) are absent, or they are different.
>
> Otherwise, classify as an overwrite.


geometry is not required anyway because we cannot force people to provide a GPX track or draw the route on the map.

elevation/height/duration: why bothering users with those attributes. They are indeed useful but we can live without them or add them later. Ditto for ratings, that were not even required before my change.
I think we should avoid required fields as much as possible.

In addition it is not a big changes to add them back in the requirements if eventually really needed.


The <dfn data-lt="message origin">Web NFC message origin</dfn> is an ...
For non-Web NFC message NFC content, the origin equals null


What are the benefits of doing it this way rather than passing these into the `Game` on initialization? (i.e. `Game.new(player1, player2)`)


The code here looks quite convoluted to me. I'd suggest by starting with a check whether there's another connection.


Ah, I missed that script. I'll see about that.


我一开始根本没联想到huffman编码，是看了解答才知道的。


@pbexe... :angry:


The realize signal receives a reference to glarea as an argument of type `Widget`, but it doesn't seem like there's a way to downcast this to a `GLArea` to invoke `make_current`, short of extracting the underlying pointer and constructing a dummy GLArea wrapper around it which seemed wrong, hence this solution using Rc. Is there a better way to do this?


@developius :::PPPP


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 48](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=412&buildTypeId=XbimEssentials_XbimEssentials) is now running


@pbexe get back to school or fix it when you get __home__ where you can __test__ and ___then___ commit


It's not clear to me what happened here, but I'd be interested to hear.


@developius But thats now fun


@PacksGamingHD This TFM compiles 'as is' with no issues. You do however require the correct dependencies to be installed. Remember that this mod is _untested_ and it's almost certain that large portions will not function, hence the [Bleeding] tag.


@pbexe yeah :)


@green-coder tks :)


@pbexe that's gr8 - now leave it alone :stuck


This doesn't work and it's screwed up all releases of messytables https://github.com/okfn/messytables/issues/144


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 48](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=412&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:04


```
In case the <dfn>NFCPushOptions.ignoreRead</dfn> property equals <code>true</true> and the <a>NFC device</a> in communication range is a <a lt-data="NFC tag">tag</a>, then the <a href="#steps-push">push algorithm</a> skip invoking the <a href="steps-receiving">receiving and parsing steps</a> on the <a>NFC content</a> read.
```


OK!


In general, stubbing methods on the class you're testing is considered a smell. Is there another way to write this test that avoids touching the `Repl` class?


@developius Ok, bye


There are other occurrences. I'm creating a ticket.


OK


You have to add an `else` statement here to catch the case of a non supported backend


Thanks! Going to a release with that soon enough.


You can keep the line but please remove the tab and spaces from it


@aancel there is a big problem with this new cmake module

The test now runs. Fine but the thing is that the headers and libs are set to miminal version of petsc....
which is not really what we want if for instance we want to use the newet preconditionner.

We should add:
 set (petsc_includes_needed ${petsc_includes_all})

and also:
    resolve_libraries (petsc_libraries_external "${petsc_libs_external}")
    foreach (pkg SYS VEC MAT DM KSP SNES TS ALL)
	  list (APPEND PETSC_LIBRARIES_${pkg}  ${petsc_libraries_external})
    endforeach (pkg)


```
If an NFC device <var>device</var> (in this case an NFC tag) comes within communication range, and <var>target</var> equals "tag" or "any"...


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 49](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=413&buildTypeId=XbimEssentials_XbimEssentials) is now running


Yep, looks good.


ファイル・フォルダの共有機能でcurrent_user以外にもid指定で共有させる状態にしちゃってます。。
Dropbox本体だと、共有用のURLを作って、そこでシェアしてる感じですが、
そういう感じに別のモデルを作って他のユーザーに共有させるようにした方が良いですかね・・？


I'm afraid I don't know of any others sorry..


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 49](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=413&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:06


fixed it! It turns out that the : was causing problems, I just had to surround the text with '' to avoid it being intepreted


that's great, but it prevents a standalone boot2k8s to work. k8s won't start


They should be included already if you have opencv on your computer.
On Nov 4, 2015 12:26 AM, "CrisAdam" <notifications@github.com> wrote:

> Can you add the referenced include files?
> #include "opencv2/video/tracking.hpp"
> #include "opencv2/imgproc/imgproc.hpp"
> #include "opencv2/highgui/highgui.hpp"
>
> and/or anything else necessary to compile the code
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/szwally/UM-Dearborn-Mars-Rover-Challenge/commit/b855309f2befa305d9d2f49014c2fccbf323cc90#commitcomment-14174016>
> .
>


@pbexe you can continue if you want to, but we don't want 1000s of commits :stuck_out_tongue_closed_eyes:


I think the problem is that the README.md is not installed unless you happen to use `-e` in the `pip install messytables` which is meant just for devs.

Can this be fixed ASAP as it is breaking our CI and no doubt everyone else who uses messytables.


I think it's clearer like this:
```
            is_conflict = (db_entry and (    self._is_upload_pending(relpath_u)
                                         or
                                            (dmd_last_downloaded_uri is not None and
                                             db_entry.last_downloaded_uri is not None and
                                             dmd_last_downloaded_uri != db_entry.last_downloaded_uri)
                                         or
                                            (dmd_last_uploaded_uri is not None and
                                             dmd_last_uploaded_uri != db_entry.last_uploaded_uri)
                                        ))
            if is_conflict:
                self._count('objects_conflicted')
```


Слияние шаблонов на мастер


ARgh you are talking about both in same sentence - that is why it is hard to follow.

When an NFC device comes within communication range verify the following conditions:
- if the device is an NFC tag, target much match "tag" or "any"
- if the device is an NFC peer, target much match "peer" or "any"
- this@[[\suspended]] is false
In case of success, run the following sub-steps


Fix this


你添加 的部分，格式调整一下。否则影响代码阅读。


In general, looks good to me but you have to address the comments before merging it.
You also have to improve your commit message. We usually starts it with a verb in imperative. Something like `Add reviewboard backend to gerrit section` might work.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 50](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=414&buildTypeId=XbimEssentials_XbimEssentials) is now running


ebcause it sounds like you talk about state instead of its value


I disagree with that... I would argue the opposite. It feels like you add a branch where there is none.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 50](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=414&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:18


Без приведения типов никак нельзя было обойтись?


Oh we did?

Is this common practice in other specs?


From: http://atastypixel.com/blog/using-remoteio-audio-unit/


Did all this test setup feel painful? I notice you need to configure side effects like making the `Repl` print an empty string and fake state, like the multiple calls to `Board#tie`. When it feels painful, it's usually a sign that the design can be simplified and the parts are too tightly coupled.


Trying to go for more readable language


Consider to specify [vagrant versions](https://docs.vagrantup.com/v2/vagrantfile/vagrant_version.html) and [box version](https://docs.vagrantup.com/v2/boxes/versioning.html), for example:
```ruby
Vagrant.require_version ">= 1.7.4"
config.vm.box_version ">= 1.0.0"
```


Эти две переменные уже проинициализированы несколькими строчками выше


自分の子供フォルダ、という意味で、インスタンスメソッドの方が使いやすいかな？と思ったのですが、
（引数でidを渡すより、オブジェクトが自分のidとusesr情報を使って条件を絞る方が自然なような気がして・・）
そういう場合でもscopeの方が良いですか？


#8 reflected.


bixão


Вероятно это должно быть в конструкторе фабрики, или в её методе init.И после этого должно запрашиваться у фабрики по мере необходимости.


@nicklockwood  will this work with remotely fetched images since the style attributes are not supported?


A lot of cleanup is needed but hey, it works :+1:


Oh, except that this still doesn't match the spec (and independently, I'm not sure that the spec is correct).


WHAT???!!!


HI Alexey,

Yes, these changes are in the newest SNAPSHOT. You might have to update your Maven cache.

Cheers,
Max


I am asking because they are marked as required in the table.


Fixed on PyPi.


Resolving it is sort of okay. But with non-primitive values it becomes broken. Not immediately sure what a better way to specify this would be. But assume you need to write the JavaScript code that creates all these objects and you can only use postMessage() to cross threads. That kind of gives you an idea as to what the constraints are.


Any ways this could go wrong?


Sounds good.


This is a nice UI touch. :+1:


Ah, good question. Except for this particular case, I don't have a strong feeling for when to use Notation instead of Definition. In the general case, when it's simple enough as below, a Notation has the advantage of avoiding unnecessary unfoldings. But in this particular case we really need it to be a Notation. The problem is the following: right now when the interpreter of MetaCoq finds a let-in expression `let x := reduce r t in u` it uses the reduction routine specified in `r` for reducing `t` before substituting `x` for the reduced value of `t` in `u`. For that to happen, the head element of the definition *must be* `reduce`. If we hide it under a Definition then it will not work. Perhaps we can do an unfolding of the head element to see if we get to a `reduce`, although that will incur in a performance penalty. What do you say @yurug ?


I think I see what you mean. This test does not check that if the computer has a choice between blocking and winning that it will try to win instead of blocking.

I think  a better this might look something like this:
```
board = ["O", "O", "X"
         "X", "O", "X"
         "6", "7", "8"]

result = game.get_best_move(board, "X")

expect(result).to be 8

```


Заменить на именованную константу, например, с помощью enum.


Заменить на именованную константу, например, с помощью enum.
```
else if (action == CHANGE)
```
Тогда и комментарий не нужен :)


Yeah it did blow up. When I tried to run it, I realized I had the wrong syntax. It should be `attr_reader :board`


Instead of constructing the url, we should get the value of the Location header returned by the POST request used to submit the job.


Hi, am a bit puzzled by this commit..

These sensor types were already getting zeroed out after 3 seconds of no movement (12 messages @ 4Hz) by the incrementing nullCount values, which I think was a more straightforward approach.

I must be missing something, but not sure what it is?


Ignore all of this, im just going nuts!


I know, but something goes wrong with the installation: symlinking, subdirs etc. I haven't figured it out yet


Would a better name have been `check_board_rows`?


Неиспользуемые переменные? Зачем они?


Why do you need jaxrs-impl for testing?


"Catégories" à remplacer par ${LangLoader::get_message('categories', 'categories-common')}


I never knew about the triple dot in Ruby ranges!


Tú si que estás entregado


I didn't understand why this was neccessary. But it seemed harmless.
Perhaps Vianney can explain what causes the problem and why this is the right solution?


I know but I don't find it relevent and maybe it was related to the "tronçons".


There was mistake here. This is my first comment


test


Isn't this a bit dangerous and maybe even wrong? This function should only be entered once during boot, so it might not be a problem? But if we would enter this function again, then the `core_mmu_find_table` in the lpae case will try to use a negative (wrap over) address (va < va_base).


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 101](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=415&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


good work @fefifofum  i guess this will fix the fb kernel panics which were seen at boot :+1:


A WriteHttpBody overload is pretty weird. What does it actually do that's different from appending the bytes to the formatter?


padding = spațiere?


It makes it dependent on the specific type of array being used. For instance if the board were changed to an array of arrays these methods would no longer work. I kept it because I thought it was very clear what was going on. If it did break, I thought it would be clear why it did and be an easy fix.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 101](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=415&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:00:41


https://github.com/dotnet/corefxlab/blob/30e99363449166542084ea2197a25e7cf97fc773/demos/LowAllocationWebServer/IFormatterHttpExtensions.cs#L46

Nothing. Can we remove that and just call Append?


Nu știu dacă ar trebui tradus ăsta, e un site numit devtollschallenger.com


Yes the code works. This is what caused the break I mentions on IRC where the Manage HL7 Errors link was not working


そうですね。逆に指摘されて私も気が付きましたが、この場合は確かにその方が自然な書き方だと思いました。


Aww. I called it that because I didn't want anyone to notice. :P


Din nou, nu cred că trebuie tradus „DevTools Challenger”


Won't do this one, because `githubUrl` is a complete github Url like `https://github.com/codebendercc/DynamicArrayHelper-Arduino-Library` and as a result, it take much more effort in the frontend in order to properly encode such a Url.


这里面并没有需要 that 的地方呀.


"Besoin d'aide ?" à mettre en variable de langue


The `null` check is already handled by `NameTypeValidator#isNameValid`.


Reading the changes more closely I can see this is NOT harmless as it removes the original code for nulling values (which is in line with the ANT spec).

Unless we get a good answer I may revert this commit until we can decide it is the right thing to do.


Yeah it looks I made that change and forgot about committing it.


thanks for catching this, bradley.


Лучше давать значимые наименования переменных, иначе магическая константа так и остается магической константой, просто вынесеной в private static final поле.


For the Archive list link it is named hl7InArchives.htm. I thought this was a convention


The file was added though the real nessecity


fixes #5


dá uma olhada no método `String.IsNullOrEmpty`


It took me a while to understand that a `PipelineConfig` is treated as a collection of `StageConfig` and therefore `isEmpty()` makes sense. Could we alias `!isEmpty()` to `hasStages()` instead?


Done


do not like


We shouldn't use `codeforgood.mit.edu` cause we're already using GitHub pages to host [the main website](https://github.com/codeforgoodmit/codeforgoodmit.github.io/blob/master/CNAME) at that URL.


Added Chestnuts.


Se poate evita elegant aici articolul hotărât și aș fi pus „peste tot” („oriunde” nici măcar nu figurează în dicționar pentru „everywhere”)
„Ia Firefox cu tine peste tot”


Also this may read better if we had a `isNameInvalid` that does the negation.


Какой глубокий смысл в этом методе? Почему нельзя было написать прямо в вызывающем его коде все что нужно?


The block now randomly spawns on the canvas and holds that position until restart. The jquery ui was added to the folder and updated with the git push because i was looking at using jquery dialog.


Can we remove this condition by setting times_seen to 1 when the `JobStatusHistory` is created?


@dkayiwa should I leave it as hl7InError?


очепятка.


scopeに分ける・・ということは、whereとnotで分割して2個scope書く？ということでしょうか？？
（どうやって直したら良いのかよくわからないです）


Should this is "/run" instead of "/root"?


And make this `nullable=False`.


nu e cumva „și sub Notificări” ?
Nu cred că în română e corect să punem virgulă aici.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3547](https://teamcity.mesosphere.io/viewLog.html?buildId=68019&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:28


Did you test it to see if it works?

On Wed, Nov 4, 2015 at 3:58 PM, Geoffrey W Wasilwa <notifications@github.com
> wrote:

> @dkayiwa <https://github.com/dkayiwa> should I leave it as hl7InError?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geoff-wasilwa/openmrs-module-legacyui/commit/a7aedbea491fe5d0d37fafd2b19ef4ee6178b82f#commitcomment-14181768>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


meeeep!


Is it really 3 times? Original PR description doesn't make it clear. 3 times sounds about right - still allows to insert blank line without losing auto-indent, and not too many times to make it weird. (Actually, it's pretty per PEP8/Python best practices, where functions are separated with 2 blank lines (won't work so well for class methods, but well, you can't have it all - it's still a REPL, not IDE)).


TeamCity pull requests :: netty [Build 583](https://garage.netty.io/teamcity/viewLog.html?buildId=2981&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5643, ignored: 21; resource leak detected (new) Build time: 01:28:38


Please remove whitespace between the method name and the parens.


Often it helps to separate out the iteration from what happens on each iteration but in this situation so little is happening that I thought it would be more explicit and clear to have it inline rather than extracting a method. Would an `empty_spot?` method be better here?


This conditional could be removed if times_seen is not nullable.  Times_seen could be unconditionally reported in the result dictionary.


Глобальный вопрос. Откуда такая любовь к отдельным приватным методам, используемым только из одного места (т.е. reuse нет), и с относительно небольшой обозримой за один проход логикой? Увеличивается ли понимаемость?


I gave the spec a good hard staring-at, and it appears to be correct. The case where the local file doesn't exist at all is classified as an overwrite in step 2a. If it does exist but the local db entry does not (which can happen if the file didn't get uploaded yet or if the upload failed), then "the last-uploaded statinfo is absent", so that's classified as a conflict. That is right, because it can only happen when there are local changes that the remote version couldn't have taken into account.


Please use the class name directly, no need to make it a string, see how we use `JettyWeakEtagMiddleware` on the line above?


May be this message can be little bit better. Say "%s is already mounted. Skipping mount". Or may be something else which is little more informative.


@bspaulding i'll merge this without further explanation, but do you care to explain this in a bit more detail? or do you prefer that i read the url? either is fine, at first glance -- and admittedly having not been in animated gif land for some time -- i'm not totally following.

does this concern you? (emphasis mine)

> Note that this block is labeled as "optional." **Not every GIF has to specify a global color table.** However, if the global color table flag is set to 1 in the logical screen descriptor block, the color table is then required to immediately follow that block.

[source](http://www.matthewflickinger.com/lab/whatsinagif/bits_and_bytes.asp#global_color_table_block) (thanks for the source!)

tests are passing so i think we're okay, generally. but curious for a bit more input.

and was the DELAY_TIME stuff i had in there just super extraneous? cool you were able to cull that out!


Btw, I now made it conscious why I was surprised with having "===" for paste mode. Paste mode makes it easy to paste from another source to uPy REPL, but complicates the opposite direction. (Just FYI, not proposing to change anything.)


perhaps bad to use the name Ymin here? perhaps instead Ylimit?


thanks a lot for this!


I will do that.


Yeah I should have added a test for this for sure. I did it as a guard against going through the whole method if there was only one space left.


remove ;


Load Balancer


codeforgood.mit.edu/heartify?


That's brill, thanks @brew


„Firefox a livrat notificări pe ecran paginilor web deschise pentru ultimele câteva lansări”
Sau „versiuni”.


Please read documentation on rake routes. `resources` is a shortcut for what you're declaring in the block.

Just having `resources :pipelines, param: :name, only: [:show, :update, :create]` is enough.

You'd put something in the block to add additional routes that are not provided by the usual CRUD verbs.


Yeah I change the name a couple commits later


How about just dropping the version there?


This is syntatically incorrect, what you had earlier was correct.


@fzurita this is causing the autosave to not being loaded and also leads to a crash when exiting. Reverting it for the moment.


good job!


I think you've missinterpret this..
Ci(Y) i would say is Cb or Cr, so not C .* Y but just C
Same thing with Ccenter


ai și „sites” după „site”.
„aceste site-uri vor fi capabile, de asemenea, să folosească acest mecanism Push.”


agreed.


Great Change


Finally adding spaces before the opening squiggly brackets I see ;P


This means I'll have to update my Weekend Quest NPC?

http://herc.ws/board/topic/9227-weekend-quests-npc/?hl=%2Bweekend+%2Bquest or can I still use integers?


The names are taken directly from the equations of "Face Detection in Color Images". I'd say that is the best solution. Feel free to change it!

(Since the equations of the two cases (`Y<Kl` and `Y>Kh`) differ, they are in separate equations.)


I ended up testing what gets printed out when I extracted the Repl class. I could have stubbed puts and checked that it received "Please choose an empty spot" as an argument.


Most of this commit seems bad formatting, it seems your IDE is having a bad day :) Keys are defined as `key: 'value'` and not `key : 'value'` — without the whitespace before the `:`


I did, and just repeated it to be sure and it is working. And it is, got rid of my initial error


```
                  'Error: advertise l2vpn evpn value does not match set value')
```


You can use interfere

Constants are for better readability


That's good work Andrea but you can't use a body tag like this. Change `body` to `span` and all is fine.

Except that your split between 1 item and multiple will only work on first page load. If you have two items and delete one, it will say "Here are the 1 thing you said you would do"—which is less than correct…

To make it work properly, you could extract the text to a partial and replace the whole text, not just the count.


OK, I'll give it a try.


Interesting, moving the location of the toast message is causing those issues? I wonder if the callback is happening in another thread.


```
           'Error: feature nv overlay evpn is not enabled')
```


is this fix going to be applied to older release versions as well? or  just going forward?


I have fixed the bad formatting in the next [commit](https://github.com/jyotisingh/gocd/commit/8f14ddaca9ab15ac6b5ee393417955be0cf66d5e). :)


moveでやっているのが、new, editと同じような感じで、モーダルを表示するだけなので
（フォルダorファイルを移動させるためのモーダルを表示する部分）
実際の更新処理はupdateを使っています。
なので、getのままで大丈夫な処理をしているだけですが、moveという単語の選択があんまり良くないですかね・・？
（自分でもパッと見たらこのメソッドで更新の処理をしてる、と思ってしまいました）


Это какая-то не очень Factory. Она еще почему-то меняет объекты, которые идут на вход. И вообще у неё нет методов get


I didn't spend much time learning about the context here, so take everything I say as very general theory, and feel free to ignore it if I'm totally off base.

Do we have to worry about lock contention since we are preventing reads while the lock is retained by Process A? Would it be possible to use an optimistic lock that only fails on step 4 when there is a conflict? To put it another way, when Process B tries to save Z with rev R it fails, meaning that Process B must re-read Z with rev R' before commencing with the save.


Here you could use:

      'fields': DEFAULT_FIELDS + [
       height_max, height_min, ...
      ],


It is Pivotal Software, Inc these days.


А мы правда должны каждый раз получать формат извне?


А мы правда должны каждый раз получать формат очищать?


Add a comment to this one like so:

```
  # This is a multi-provider, enable-only feature. Do not disable from provider.
```


Whitespace nazi, angry. Whitespace after `||` pretty please?


I suspect that we don't have this unified across all sub-projects but it should simply say "is RabbitMQ."


``page_path`` or ``page``?


丸亀国際ハーフ通過は公認記録となる


My auto saves are still being loaded correctly when choosing the resume option. Under what circumstances are auto saves not loading correctly?


No problem. Anyway, thanks a lot for highlighting the bug


does this work? i mean, can i pick up arrows without crashing?


OK, in progress :)


> also and isn't this just the minimum equation and not maximum?

Yes, that is correct. You're commenting on a commit that (by my mistake) doesn't rename the maximum function.


A test to make sure that `valid_spot?` worked would have been good here. I ended up testing user input later on.


deleting this premature commit , forgot to change the name of the fork


Sure, will do that.


ありがとうございます！(すいません、手元で rebase して作業してしまっていたのでこのコミットは消えちゃいました…。


他人のファイルやフォルダも共有できるという仕様だったんですね。
仕様上あえてそうしているのであれば問題はないと思うのですが、自分のファイルやフォルダを共有できるのは自分だけという仕様の場合、セキュリティ・ホールになるコードだと感じました。

共有できるのは自分のファイルやフォルダだけ、という仕様だと仮定して、自分であれば FoldersController と FolderFilesController に share アクションを追加して、厳密に current_user から フォルダー・ファイルを引いた上でSharingFile もしくは SharingFolder オブジェクトを作る処理を書くと思います。

もちろん、私のやり方が正解というわけではないのですが。


Now it is looking good.
We can remove `osm_amenities`  as we don't use it in any query.

![table_layer_diagram](https://cloud.githubusercontent.com/assets/1288339/10938947/55ad01c0-82fe-11e5-8309-070b162a189a.png)


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


Didn't know this was supported. That's awesome!


なるほど！ Box note みたいなやつですね。多分。


Please leave white space above all the comments.  If you add a new line of code then there should be a blank line between that line and the comment to allow the comment to stand out.


@lucabrivio I still miss the logical reason why the best score should be the correct one to output. What if we are going to lose, and the 'correct' score is even worse than that of the main thread?


Il y a pas de faute d'orthograp^he ni de grammaire


Just starting the app and resuming a game will produce the issue for me
Propably a race condition on my particular device.


the part that i can't get my head around is where the `packed` byte is a big value allowing us to calculate the table size!

i'd expect that buffer.slice(10,11)[0] to be the `global color table flag` -- ¯\_(ツ)_/¯

![logical descriptor](http://www.matthewflickinger.com/lab/whatsinagif/logical_screen_desc_block.gif)

tests are good, so let me know if you have thoughts or clarifications. i'll merge this.


That is `git mv`, not what we want.

I would like all your commits to never mention `testly`, only the cooler version. There should be no commit for the rename. You need only a commit to change the name inside `__openerp__.py`


I see that now :+1:  :)


Why the default in the namespace if we get one error page for each code?


Ok, sounds that way. I really doubt that the toast message is the root cause. It sounds like we need to synchronize something.


Hej!


This does not seem to work for me.


Hey,

I have cleared my cache while I was waiting for your answer. Still getting this error. But after looking into the snapshots repo with a colleague we've found that [flink-storm](https://repository.apache.org/content/groups/snapshots/org/apache/flink/flink-storm/1.0-SNAPSHOT/) is from 23 October, while [flink-streaming-java](https://repository.apache.org/content/groups/snapshots/org/apache/flink/flink-streaming-java/1.0-SNAPSHOT/) package is more up to date (3 November) as I am getting right ``StreamingGraph`` version and wrong ``FlinkLocalCluster`` version.

Hope this is a the reason for my problem and you could fix it. ;)

Cheers


:beers:


When I was making those changes, it did not occur to me that it could break, but of course it could have. A test of the messages would have been better because when I did move them into the constant, I could run the tests to make sure that they were still printed out.


Would `srcStage.hasJobConfigByName` read better?

Also `jobConfigByInstanceName` and `jobConfigByConfigName` look nearly identical, can we make one use the other. Better I don't see any place where `jobConfigByInstanceName` is being passed a `false` for `ignoreCase` – can we drop the method altogether?


Consider using Google Calipers for micro benchmarking ?? - https://github.com/google/caliper


This error was happening because you removed `@items = @user.items` from the controller… so it's no longer available to your view. In any case, doing `@user.items` in your view is slightly better.


Done, changed it in all json responses


If you mean where there is a tie, I think I added a test for that later.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 51](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=416&buildTypeId=XbimEssentials_XbimEssentials) is now running


Key is dead my friend.
On 11/03/2015 08:19 PM, David Mohl wrote:
> Yeah, now I can finally mess up your wakatime ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robbyoconnor/.spacemacs.d/commit/352d05c4f5869a75fe8ebbaec671a9dd04029cfe#commitcomment-14171537>.
>


Good point @smarthi. I think nobody so far considered it. Should take a look.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 51](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=416&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:11


Add .ts files as well


You could perhaps validate with a custom validator on update. But it's simpler to just check the validity of the current password in your controller. If it's not valid, you should render the form again with an error. Otherwise it goes through and updates the model.

You can reuse the `User.authenticate` method for this.


Wouldn't it be just as easy to merely delete the template file?


You should preserve authorship here.


格式


Like I said before, size is already a multiple of the page size here.


Still dont understand how we should use equation 7 and 8 here with Kh since Kh is outside of their limits?


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


Hey, you removed LWJGL as a dependency??


I don't understand the reason but tests so far have been showing it works, at least to some extent!


Move this to `test_selector.py` and rename it `test_dsl.py`


Why add this as it does not need to to support additional transports?

Should this not test that the connection object is working and has the attributes & event emitter?


I guess you're right, I always hated those toast btw :-P


Yes, it's 3 times: 1 to finish current line, 1 for the blank line, 1 for the completely empty line to exit.  (So it's actually 2 times if you already finished your current line).

You can't actually insert blank lines with this patch.  Allowing that would make it 4 enters to exit.


@rholder I think this is a **bug**
in line 303 above,
double exp = Math.pow(2, failedAttempt.getAttemptNumber());
should be:
double exp = Math.pow(**this.multiplier**, failedAttempt.getAttemptNumber());


Le message aurait dû lire "Les IDs sont insensibles à la casse."


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


really nice commit !


> Paste mode makes it easy to paste from another source to uPy REPL, but complicates the opposite direction.

That's true.  But if you already pasted from some other source, then likely you still have the source to re-paste from.  The use case would be typing code in an editor and constantly pasting it across to uPy.


An options hash would probably make it more explicit what is true or false. `{human: true}`. I think I make similar mistake later when I extract messages.


Odd that this says connect to the default host yet the default host is in fact `realtime.ably.io`?


In Mac OS X is a nightmare to mix SWT with Swing due to underlying changes in the UI threads and in general it's not recommended.


Key string has to have a delimiter(\e) between key and hname. see https://github.com/roma/roma-ruby-client/blob/master/lib/roma/client/rclient.rb#L403


@jarodreyes You can just source this .env file as in

```bash
$ source .env
```

But yes, I'll add this to the steps.


Change to be 'app/test/{spec,mock,e2e}/**/*.js'


@antonpirker thanks! should we add "ext in [".jpg", ...]" to the if statement or remove the RGB conversion altogether? any opinion?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3551](https://teamcity.mesosphere.io/viewLog.html?buildId=68028&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:12:21


This needs to be fixed.

        logger.error('Unable to create config directory: {0}. Exiting ...'
                     .format(config_dir))


estas linhas sobrando podem ser cortadas :scissors:


Verified this is working. Ignore my feedback from earlier.


@Chive you missed to adapt the changelog


You cannot use `setq` here, as it only sets the pointer to the array and does not copy the whole array.
Instead, you need to go in a loop and set the values. E.g.:

(defvar *initial-world*
  #2A((w g g g g g w g g g w g g g)
      (w g g g g g w g g g w g g g)
      (w w w g w w w w g w w w w g)
      (g g g g g g g g g g g g g g)
      (w w w g w w w w w w g w w w)
      (g g w g w g g g w g g g w w)
      (g g w g w g g g w g g g w w)
      (g g g g g g g g w g g g w w)
      (g g w g w g g g w w w w w w)
      (w w w g w w w w w w w w w w)
      (g g w g w g g g w w w w w w)
      (g g g g w g g g w w w w w w)
      (g g w g g g g g w w w w w w)))

  (dotimes (x (first (array-dimensions *world*)))
    (dotimes (y (second (array-dimensions *world*)))
      (setf (aref *world* x y) (aref *initial-world* x y))))

(-1 point)


> 自分のファイルやフォルダを共有できるのは自分だけという仕様の場合、セキュリティ・ホールになるコード
なるほど！その点全然考えてなかったです・・
共有された側からもファイル・フォルダが見えないと・・という方だけ考えてたので、全体から探せないとだから、仕方ないかな、と思ってたのですが。
共有を作る時はちゃんとcurrent_userから辿らないと！ですね。
（見るだけ、と共有するがごちゃ混ぜになってました）
この辺もうちょっと整理して見なおしてみます！


Make sense


Pq não coloca isso na action da controller?

Desta forma vale mais a pena deixar fixo na tag `title`, é mais performático:

```
<title>Relatório</title>
```


I see. Then we should have it, especially as most models can be easily exported, so why not also trees? The code looks very reasonable :+1:


Am I missing something, but these tests aren't checking the key or values of the params so I am not really sure what value these tests offer?


Como te dije, podes reemplazar gtk_window_get_titlebar' por 'gtk_dialog_get_header_bar' :wink:


thanks, I updated the readme


Should we not heck the values are as expected?


Don't use feature detector, compile both


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


I see in your next commit you address the above, however you still don't check the values?


What purpose does this `read` have here?
There is no requirement for user input in an initialization function actually.

(-0.5 points)


Estas cambiando dflags nuevamente cuando ya no se utiliza mas.. ;)


还是可以看到，后台管理重新生成AccessKey。


the key was the order of `Attributes` and `DynamicColumns` for the `include`..


Actually, visualization should not change the state of a variable that it is visualizing, it should only print a different representation...


Most of the other loggers are using {0} for the syntax (like the logger.error just above this one).  Let's be consistent.


@azawawi you also need to fix the paths here: https://github.com/perl6/modules.perl6.org/blob/master/web/build-project-list.pl#L42

I believe when the site is being made, all the assets are copied somewhere.


:+1:  You are the best!
And all this time we were blaming babelify


Yes, so much to theory and practice. ;-)
But I think I'm going to submit a test, where I pick the thread with the highest completed depth and the highest selective depth.


The scan failed.
Included commits

Commit ID	Owner

84652d046be449561880fdd008307d81cc08a6d7	@BearSpace


Consistent use of formatting syntax .... {0} vs. +


What about a test to test the opposite condition i.e. autoConnection false.  Given `autoConnect` defaults to `true`, this test does not unfortunately test much.  The tests should really check if autoConnect is true by default, then the test you have above, plus the test with autoConnect false.


Ditto


很悲哀


Nice have to extract this on separate gem


@tenderlove Hi, I guess this is the API for Rack you talked in FullStackFest this year. Are you using it in Rails? I couldn't find examples.


Tim, do not use table layout then. Remove the description ("Paid By:", "Mode" and etc) , make the spinners and buttons width match parrent, and the hint in widget will show what the description is.

For the spinner, the first item by default is not self, it is the "Paid by:" ,then goes to self and friends.
For the button, the text inside button is "Add member(s)"
Same goes for date and snapshot.

So when you click submit, it will check through all the options: if the spinner selection is "Paid by" which the user didn't select anyone specifically, the confirm statement action will be denied, required that the user has to select who paid for it (this also prevents some human mistakes). If the member is empty, action will be denied as well.

Lastly, I would like to add a add note button, which is when you click into it, it will display a dialog box with edit text box, this is for user to make some notes just in case. (optional for user to fill)

Thanks a lot!!!


relations Polymorphic -> relations polymorphiques
n'est pas encore supporté -> ne sont pas encore supportées
Desolé -> Désolé


next step :P


See comment above


With this check the robot will not be able to collect anything if it is standing on the border of the map.
The check should be on the coordinates of the treasure (e.g. `{(- zmienna1 1), (- zmienna2 1)}`) and not the coordinates of the robot itself.

(-1 point)


style: prefer &&


You should use k instead of key.


At the time I did this, I think I had a reason, but right now I can't think of it. I also ended constructing players in the initialize method of `Game`.


(+4.5 points)


Style: foreach () {


The correctness of this depends on the resource gender.


fr


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|__suite_setup__|0|1|0|0
basic_test|_basic/test_create_vm_|0|0|0|1
basic_test|_basic/test_stop_vm_|0|0|0|1
basic_test|_basic/test_start_vm_|0|0|0|1
basic_test|_basic/test_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_volume_|0|0|0|1
basic_test|_basic/test_add_vol_to_stopvm_|0|0|0|1
basic_test|_basic/test_add_volume_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_multi_volumes_|0|0|0|1
basic_test|_basic/test_add_volume_negative_|0|0|0|1
basic_test|_basic/test_crt_temp_from_volume_|0|0|0|1
basic_test|_basic/test_crt_vm_with_volume_|0|0|0|1
basic_test|_basic/test_vm_securitygroup_|0|0|0|1
basic_test|_basic/test_create_iso_vm_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account2_|0|0|0|1
basic_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
basic_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
basic_test|_suite_teardown_|0|0|0|1
virtual_router_test|__suite_setup__|0|1|0|0
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
virtual_router_test|_novlan/test_create_vm_|0|0|0|1
virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_vlan/test_create_vm_|0|0|0|1
virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_snat_create_|0|0|0|1
virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
virtual_router_test|_vlan/test_start_vm_|0|0|0|1
virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
virtual_router_test|_volume/test_add_volume_|0|0|0|1
virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
virtual_router_test|_sg/test_ingress_|0|0|0|1
virtual_router_test|_sg/test_egress_|0|0|0|1
virtual_router_test|_sg/test_icmp_|0|0|0|1
virtual_router_test|_sg/test_udp_|0|0|0|1
virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
virtual_router_test|_sg/test_vm_ops_|0|0|0|1
virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
virtual_router_test|_sg/test_detach_l3_|0|0|0|1
virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
virtual_router_test|_pf/test_attach_pf_|0|0|0|1
virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_multi_rules_|0|0|0|1
virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
virtual_router_test|_pf/test_tcp_|0|0|0|1
virtual_router_test|_pf/test_udp_|0|0|0|1
virtual_router_test|_pf/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_2_eips_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
virtual_router_test|_eip/test_attach_eip_|0|0|0|1
virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
virtual_router_test|_eip/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
virtual_router_test|_lb/test_create_lb_|0|0|0|1
virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_suite_teardown_|0|0|0|1
local_ps_virtual_router_test|__suite_setup__|0|1|0|0
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_ps_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_ps_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_udp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_ps_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_ps_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_ps_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_udp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_ps_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_ps_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_ps_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_ps_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_ps_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_suite_teardown_|0|0|0|1
local_nfs_virtual_router_test|__suite_setup__|0|1|0|0
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_nfs_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_nfs_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_nfs_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_nfs_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_suite_teardown_|0|0|0|1
Total|304|0|4|0|300


You should add delimiter(\e).


"May" or "must"? :-)


Duda... Estas usando el plugin lyrics?????

Estas opciones están puestas así para que ambas barras laterales puedan redimensionarse correctamente.

Por favor comprobá tu cambio con ambos paneles activos y con desactivarlos. En particular, también tenes que comprobar que cuando pragha se inicie por primera vez (Básicamente eliminando la carpeta ~/.config/pragha), los paneles se inicien y muestren correctamente.


Is the system `url` package not being used here for a reason?


Please rename to be "out" and not "test"


The "path/row line" needs to start on a new line (that's why it's indented by 4 spaces) and the same goes for the "years line".


it's not necessary to wrap this is parens


I cannot give you points for this assignment until you make the repository private (https://education.github.com/discount_requests/new). Did you get into problems with this or did you not try yet?


自分であれば、今の move に相当する処理は js ( coffee ) だけでやってしまい、 move アクションでは実際にファイルやフォルダを移動する ( parent_id を変更する? ) 処理を書いたと思います。

```coffee
$('js-move-button').on click, ->
  $('.js-modal').modal('show')
```
みたいな感じですかね。クラス名などは適当です。

update というひとつのアクションに、 「移動」と「変更（名前など）」 の２つのアクションが入っていると危うい感じがしました。
move も update も根本的にには同じ処理である事は間違いないのですが...


Only needed if your statement spans multiple lines: https://github.com/airbnb/javascript#8.3


> You can't actually insert blank lines with this patch. Allowing that would make it 4 enters to exit.

Oh, so then I'd say it's a bit of issue. I don't know if I want to suggest to suggest making it 4 enters to exit (partly because it may be hard to check for; though specified as "pressing enter on 2nd blank line to exit, it's not that bad from UX perspective").


Tamam abi anladım demek istediğini.


Can you explain why the URL would be different? and why it should err? It's also probably better to include this in a different if statement since URL not being equal is not the same as `'Status must be 200;`


Unneeded dependency. Might as well delete it as you're making other changes.


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


new


It was already included in line 11


This indenting is not good.  Keep it consistent.

        logger.info('Spawning {0} scenes for boosted regression via {1} '
                          'processors ....'.format(num_boosted_scenes,
                          num_processors))


So this is a management API change, not just xml?

Also, the text descriptions still refer to connection limits.


it is not available if default libraries were used


Why using protobuf at storage level? It's a simple structure with 2bytes and you could avoid yet another copy here.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3553](https://teamcity.mesosphere.io/viewLog.html?buildId=68040&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:59


e o arquivinho CSS aquele?


Coloca esse estilo em uma classe css.


I think I commented these tests out because they were no longer relevant for `Game`. A lot them were being moved to `Player`.


You need to include full LICENSE here. It is not enough to point to license file.


I think this function would be clearer with the two lists -- the down-levelled hosts, and the hosts causing pool down-levelling -- pulled out first. Then just call one of the three permission functions according to which lists are non-empty.


were


> That's true. But if you already pasted from some other source, then likely you still have the source to re-paste from. The use case would be typing code in an editor and constantly pasting it across to uPy.

D'oh, right! I meant following usecase: you do explaratory hacking in REPL, and then think: hey, that's the function I'd rather save for reuse! Indeed, it's not related to paste mode. And yep, that all hits hardly on having some external shell to do those tricks, not something to dump on REPL (current workaround of pasting it into editor which has column blocks, to cut ">>>"/"..." should be pretty acceptable).


Nice set of unit tests! :smile:


update themo


$gd_version_number now is the actual version, or 1, even when the extention is not loaded


No it's not. It's updating us to use the correct names in 1.x, rather than the old aliases. It's totally unrelated to 2.x.


I don't think you're using Linq in this file either.


cc/ @BLangendorf for review


Is the CPU vendor not tested earlier? If not, it would probably be cleaner to pull it out into a separate test anyway.


How do we know this is #000000?


Same comment about testing the vendor separately.


You're the best, thanks


Aslında hepsini bölerek ts arrayi içine atmış oldum. Örneğin ts[0].tokensave = 'A', ts[1].tokensave = 'Ali', ts[2].tokensave = 'Aksu' gibi. Elimde parça parça her şey var aslında ama bu inputları nasıl kullanabileceğimi çıkaramıyorum


for disaster commit to master


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


Why is `alias` package for apache needed?


This isn't a good name, and it doesn't follow the other ones, for example the one above it. I'm sure you just got the TBO one, but still, doesn't follow the style. :P It should be "editor_account_header_background_color".


It would fail if device name wasn't supplied


> Oh, so then I'd say it's a bit of issue.

Maybe have a play with the current implementation to see what you think.  The second enter doesn't auto-indent, but you can still press space to get a blank line and continue (otherwise pressing enter straight away wil exit).  It kind of gives you a warning that you're about to leave the mode.

> I don't know if I want to suggest to suggest making it 4 enters to exit

Well, it's a bit more hacking/code-size to do this, but if you think it's important...


I think (intA & intB) != intB is sufficient.


global_init_prefork already lock pidfile?


Depending on the amount of time till release, would you be willing to compile a beta?


Thanks for reply.

I can swear that I have ide configured to insert spaces instead of tabs. I don't know what happend. I'll use uncrustify  and pos changes.


「分ける」という書き方がわかりにくかったですね。申し訳ない。

```rb
scope :other_users, ->(id) { where.not(id: id) }
```

あんまり読みやすくなっていないかもですね...


This should be logger.error since it's an error message ...


I used this <code> read </code> only for debugging and forgot to comment out.


Would a test double of `gets` be better here?

If I could use something that could stand in for the user input without mocking `gets` that might be better.

I also read something about using the `StringIO` class to deal with IO operations, but I wasn't completely clear on it so I didn't use it.


@gonzomir is there a reason to use docxgen.js here and docxgen.min.js in ui-matrix.html (line 43) ?


is this the same as docxgen.js  in ui-matrix-report.html ?


most if not all from the includes above are part of common.h


This is not the correct way to enable a site in ubuntu, you need to use `a2ensite`.


el diego se la come


Remove "Error" from the string since you are using logger.error


I'm sorry. I forgot about this. I sent request 2 minutes ago.


Este siempre fue un caso controvertido,  :sweat_smile:, sin embargo no debes tocar el margen, el único que tiene problema es el GtkEntry y estas modificando todo.. exagerando espacios..


2 and 4098 should be declared as a defines


We don't need to test one character at a time; we can test sixteen characters at a time. Or eight characters at a time in a 32-bit Int, if we know that the inputs have a multiple of eight chars but not necessarily a multiple of 16.


Kh could be named K here, that would improve the clarity of what it actually is. In some cases it is Kh and in other cases it is Kl. The different cases are then separated through the binary threshold maps.


space after =


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


Ohhh I can't wait to test this :P

Since the arduino due runs ön 3.3V I'll add a pullup to that voltage instead of 5V as the document said.

Thanks.


> Oh, so then I'd say it's a bit of issue.

I don't think it's really an issue, but let's play with it as Damien said.


删除空标签


space after while


space after if


We do not use it . It's a POC so it was fastest way to do it .

On Wed, Nov 4, 2015, 15:41 Luca Garulli <notifications@github.com> wrote:

> Why using protobuf at storage level? It's a simple structure with 2bytes
> and you could avoid yet another copy here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/orientechnologies/orientdb/commit/59b5e4c722b9750108993783e709261a1a02be7e#commitcomment-14182875>
> .
>
--
Best regards,
Andrey Lomakin, R&D lead.
twitter:@Andrey_Lomakin linkedin:https://ua.linkedin.com/in/andreylomakin

Orient Technologies
the Company behind OrientDB


Leave white space above the comment.


Relevant: https://github.com/bitcoin/bitcoin/issues/6113


TeamCity pull requests :: netty [Build 584](https://garage.netty.io/teamcity/viewLog.html?buildId=2994&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4796, ignored: 21 Build time: 00:37:03


Sorry to be pedantic, but could we change LessFeatures() to FewerFeatures() throughout, otherwise the grammar police will get us. :police_car:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3556](https://teamcity.mesosphere.io/viewLog.html?buildId=68050&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:11


Why don't you guys just implement https://github.com/dotnet/roslyn/issues/1171 :smile:


This is Lauren's but i stupidly let George name it.


This is invalid syntax "..format()"


Yes it did feel painful. While writing this it became clear that `Game` knew about everything but I wasn't sure how to limit its knowledge. It seemed that for the game to work there would have to be at least one thing that knew about everything else.

I think I could have have hid the calls to board and player behind methods like:
```
def tie?
board.tie?
end
def game_over?
board.game_over?
end
```
The tests would have been easier to write, `Game` still feels like it knows too much. I'm not really sure how to have it know less.


Should be logger.error


Installing `xdebug` unconditionally (including production) is not recommended, it will slow down requests by a factor of 2 up to 10.


There are a couple of general requests.
1. Add comments above test functions
2. Change commit message to include test description and proper subject.
3. Change "infra" to be infrastructure in the commit message. It will help grepping.


As far as I could tell, there wasn't actually any modification happening to the overlay's contents, so maybe not doing anything is more correct. I think when the Emacs bug is fixed we'll be in a better position to decide.


這邊少空一格


You can get them in Preferences > Downloads > All Docsets. Make sure you're using the latest version of Dash (3.1.3).


for IE < 9 you would have to use [polyfill](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/slice#Streamlining_cross-browser_behavior)


Parece que funciona bien.. Aunque en xfce no respeta ninguna preferencia de CSD, porque el dialogo se crea automáticamente.


There is no need to inherit from NodeVisitor


I made a typho mistake of push. The repired, expanded comment is:

cfg.hideTrigger usage of http input field width calculation


這上面的縮排有些都不大對


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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3559](https://teamcity.mesosphere.io/viewLog.html?buildId=68056&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:08


I think you should add MAP_FIXED here.


It would be good to add a comment here explaining that it's "less in at least one bit" not "strictly less"


I changed this around because it sometimes would not work with the previous check but now I don't remember why it wasn't working.

This check is dependent on the space being the string representation of a number, if that changes, this would no longer work.


Ok, nevermind then. (+0.5 points)


Typo everywhere s/definations/definitions


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


Why the empty {}?  Prefer {0} to be consistent.


If we'll be unlucky we might overwrite existing mapping. I'm not sure how to avoid that...


Yeah this one was tricky to figure because the tests did not catch it and I only noticed sometimes while playing.


Excellent! Out of curiosity, why was this patch rejected by the less maintainer?


Don't like the indentation on this last line.  Bring it back to be consistent with all your other indentation.


test the upgrade in your dev environment with some production data and using the production s3 config (this step does not modify or destroy any s3 file)
once you are okay, deploy to production and run `db:migrate`


Maybe a move with higher score will do better not because it is more accurate, but because if it works, it is more likely to help gain half a point than would be a move with a lower score (and same statistical accuracy). That is, we are hacking the minimax model... and I'd like to check whether this widens the performance gap with weaker engines.
Maybe with many threads we should discard the highest scoring move, as it may be more likely to be less accurate, and pick the second best scoring one.


ok let start UPsss


Empty {}


if you want to visually show priorities of execution, please move "||" under "type" or a bit more to the right. All "&&" should be shifter to the right too on the same amount of spaces as you move "||"


If you want, I can just start commenting on all the "inspired" commits, for instance:
https://github.com/brad-accuvant/cuckoomon-modified/commit/e2d24a5f91c7670443dafbeff68f830afdf13e8a
https://github.com/jbremer/monitor/commit/2ae2175a37a601bd8a864da5b9cd2c35cdfcd063

You can act shocked all you want, but it wouldn't be an issue if you guys didn't seem to get physically ill by even mentioning when you're getting your ideas from 80% of the time.  I realize you have to play games with purposeful renaming of variables and such to make the code look like it's not a direct ripoff (and thus a copyright violation, which might be important to the people you're selling Cuckoo to under a non-GPL license), but it really would not kill you to have some decency.

I just find it interesting that since the subtlety of my code revalidating all hooks (as an unload of one DLL could also trigger the unloading of other DLLs if their refcount was 0) was missed as it wasn't mentioned explicitly, not to mention that the code was added to LdrUnloadDll as in my old commit even though you now have a proper load/unload notification hook (also from me) that would have been the proper place to put it now.  It looks a lot more like "inspiration" than someone who understands their own codebase.

-Brad


logger.error


@conorroche, please, be aware I'm not sure if the -subTypesAnnotations is the only not working param


χαχαχα και'γω νόμιζα πως καταλάθος σου έκανα pull request και αναρωτιόμουν πόσο ανάπηρος είμαι για να μην το έχω καταλάβει :-) -- ελπίζω σήμερα το βράδυ να βρω λίγο χρόνο να σου στείλω δύο pull requests


farpait


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 52](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=417&buildTypeId=XbimEssentials_XbimEssentials) is now running


It is only after [20151104071547_populate_paperclip_data_into_attache_json.rb](https://github.com/choonkeat/paperclip_to_attache/blob/a1f5d5b4cd90565dcaa3ad7cda3b348bf73c808e/db/migrate/20151104071547_populate_paperclip_data_into_attache_json.rb) migration has run
THEN we can introduce the commit to remove Paperclip, AWS gem and the S3 config


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


Also changed POST /me handler to PATCH, for user update, and POST for user creation


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


Sure? So it is in UCX...


What about testability? How can you do unit tests for Logger with stub of StateFactory?


Código muito complexo.

Pensa mais simples:

```
return xmlJogo.Elements().OrderByDescending(_ => Convert.ToDecimal(_.Element("preco").Value)).ElementAt(0);
```


:heart:


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 52](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=417&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:02


Please remove dead comments


```
- if @prototype.first_thumbnail?
```
とすればhelperいっぱい定義する必要もないのではないでしょうか


:shipit:


Could probably be written better with a while loop, but practising recursion and wanted to use it.


is this needed?


> I don't think it's really an issue, but let's play with it as Damien said.

Just think yourself: all these efforts were made to let people type in well-looking Python code comfortably. To leave auto-indent, you would need to press backspace 4-5 times max (each cuts 4 spaces). If you wanted to insert blank line, but instead had REPL leaving auto-indent for you, you may need to press space 16-20 times to reinstate it. Couple of times I have REPL do that to me, I stop trusting that it was designed to make my life comfortable, and instead will acquire suspicion that it was designed to give me unpleasant surprises when I don't even expect them, and will just try to avoid using it as much as possible => al the efforts on REPL are wasted.

But yes, indeed need to play with it - just need to anticipate what kind of problems to expect and then see if they're really that problematic.


Good for move out functionality to mapper. But what about testability? How do you test Logger with Mapper stub?


Perkins split the GET/POST into two methods--I think because it was easier for beginners to understand. I think it makes more sense to have the one method do both (DRYer!)--but the method is a little long, isn't it?


헙. 죄송합니다.. 잘못올렸네요 ㅜㅜ...


Should this not use a `List` as opposed to an `Array` to ensure uniqueness i.e. if we had two CLOSED and no CLOSING, this test would still pass


Hi @mrhanlon , when you get a chance today, could you please pull this theme css change into the araport-dev and araport-prod portals?

Thank you!


This description is wrong, these are states emitted when closing according to the spec


@tristanhimmelman this really made a difference, according to my benchmarks - thank you!!


great, and we move another http resource to https


q pajja


si se la come es su problema po wn


dejalo ser


No need to `chmod` both `build` and `build/logs` since you `-R` on a parent of it.


RTC4e actually specifies that it emits a StateChange object which includes the current state, last state, and reason


:) beat me to it


Alexandre's claim about Stockfish being much stronger than before against Critter seems to confirm this...


This is not a state change object but simply the current state.


so what exactly do we need to do now?
I think we need to do the random number generator from the lab 5 session


Uncommunicative name. Is it _default_ state rather empty state? Or emty _buffer_ state?


I see a lot of changes due to indentation. Should we agree on some coding style? I have an .editorconfig in the repo.... in case your editor supports this.


ifのendなくて大丈夫ですか？


hi,everybody there! i am working on create a operate system macos,welcome to here for revise>


Its place is in state package.


Looks like you accidentally replaced `lock_guard` with `mutex_guard` here and below.


Good point. Thanks


thanks for doing this @vzvenyach. merging.


Yeah I saw it was freaking out on the lambda.  Thanks for the tip


@alwey: I would like 20000 - 40000 games (10000 - 20000 iterations). What is your idea about tuning?


with this `shift_form.html`, one simple django template is enough for both create and update view, whereas originally it required 3 template files; `shift_new.html`, `shift_edit.html`, `shift_details.html` and those are rather complicated. Nonetheless, we may need to take the latter in the end, so I did not delete them for now.


Awesome! Glad to improve things :smile:


it's kinda sad that things like this have to be explicitly written down nowadays...
Is this a reaction to instances of this happening within xoreos in the past? or is it only a precaution?


The Master is the beta....or you mean something else?


Hi

Yes we hate tabs. If we see a tab where it should be ‘4 spaces’ like in proposed in your .editorconfig, then we kill them :)

P@

> On 04 Nov 2015, at 15:13, Vitali Peil <notifications@github.com> wrote:
>
> I see a lot of changes due to indentation. Should we agree on some coding style? I have an .editorconfig in the repo.... in case your editor supports this.
>
> —
> Reply to this email directly or view it on GitHub.
>

Patrick Hochstenbach - digital architect
University Library Ghent
Rozier 9 - 9000 Ghent - Belgium
patrick.hochstenbach@ugent.be
+32 (0)9 264 7980


Je ne vois pas comment prendre en compte le genre ?


Be consistent with {0} and {1}


Done


Should be three different tests. When Three elements array, one elements array, empty array.
But it's very good that you think through corner cases.


A po co: zmieniłem default na true w wywołaniach funkcji.


Indenting ...


Indenting


need to update mtt config as well


_Very_ large test. Should be splitted to some scenarios.
You cover several scenarios with one test. Use one test for one scenario.
XXXstate -> YYYState.


good focused test covering _one_ scenario.


Wouldn't it be better to present this as `return done(...)` ?


:smile:


So ... the file list is not available from webhooks

https://bitbucket.org/site/master/issues/11703/add-modified-file-list-in-the-repo-push

Too bad :disappointed:


%s in here needs to be {1} and the current {1} needs to be {2}


or http://bundler.io/v1.10/bundle_init.html @andrewsunglaekim


Although...isn't this really close to `g_spawn_sync()` + `strstr` ?


I agree that tabs are a thing of the past. All coding style guidelines say the same.


Oi.

Esse construtor é inútil. Nota que **TODAS** propriedades tem o modificador `set` como público. Logo tu pode usar o type initializer do C#:

```
new RelatorioModel
{
  QuantidadeDeJogos = 12,
  PrecoMedio = 45.4,
  JogoMaisCaro = "Donkey Kong Country",
  JogoMaisBarato = "Bomberman",
  ListaDeJogos = new List<Jogo>
  {
    new Jogo { Nome = "Bomberman", },
    new Jogo { Nome = "Donkey Kong Country", }
  }
}
```


Yes. I intend to support width/height styles in the near future, but for now it will just draw the image at whatever size it comes. You can set the scale by adding a scale property to your imageSource though:

    <Image source={{uri: 'http://example.com/someimage.jpg', scale: 2}}/>

You can also use that to set the size if you set the scale to something other than the actual pixel density (for example, a scale of 2 draws the image at half size, a scale of 4 draws it at one quarter size, a scale of 0.5 draws it at double size, and so on).


Try to avoid _me_ in naming. Otherwise you should use it _everywhere_ )))


:warning: :warning: :warning:

Propriedades em C# são PascalCase, e não camelCase!!


You can reply to all and manually edit the recipient in your mail,
but it's not very convenient.

I can add a '--sender' option,
that could be easier, it's something you want ?


White space between the logger line and the comment.


...sorry guys


nitpick but it seems better to have a shorter line:

```javascript
if($editable.is('[data-oe-model]:not([data-oe-model="ir.ui.view"],[data-oe-type="html"])')) {
  //...
}
```


**README - Nitpick**
You should have included more description on how to use the ORM.


@fissban Me hace ruido esto. ¿Estás seguro que se van a poder seguir agregando elementos? Según la documentación de Java (http://docs.oracle.com/javase/7/docs/api/java/util/Arrays.html) devuelve una lista de tamaño fijo.

Otra forma de hacerlo, evitar lo de arriba (si es realmente así) y que queda con menos código es:
```java
public void addScheduledEvent(EventScheduled event)
{
	if (!_scheduledEvents.containsKey(event.getTime()))
	{
		_scheduledEvents.put(event.getTime(), new ArrayList<>());
	}
	_scheduledEvents.get(event.getTime()).add(event);
}
```
¿Qué opinás?


I'd rather only generate the hash once, performance


please put a comments in Input where you expect violation


Ha ha. Lesson learnt I hope :)

On 4 November 2015 at 13:53, laurenstevens <notifications@github.com> wrote:

> This is Lauren's but i stupidly let George name it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/S6C/test/commit/ba67fcf6ba167add97aab3ac94147c3b8dc6af22#commitcomment-14183223>
> .
>

--
Best regards,

*Craig Chambers*
Network Manager

Wylye Building, Southampton Road, Salisbury, Wiltshire, SP1 2LW

Telephone 01722 597970

email: office@salisbury6c.ac.uk web: www.s6c.org.uk

--
Important Information: This e-mail is intended solely for the person or
organisation to whom it is addressed, and may contain confidential
information.  If you have received this e-mail in error or are aware that
you are not authorised to have it, you MUST NOT use or copy it, or disclose
its contents to any person. If you have received this e-mail in error,
please return it to the sender immediately.

Think of the environment.  Please avoid printing this e-mail unnecessarily.


The scan failed.
Included commits

Commit ID	Owner

a6f44e41cbec199739cdda9aba94708fd0a2405a	@amitchen22


Awesome - are we going to make this API(able) so users can adjust entity sizes for modded mobs?
I don't know if something like an entity registry file is possible that people could add other mobs to and manually adjust the sizes or if it would be a massive pain in the bum.


you'll probably want to assert below that the file's gone.


LGTM


static there will be redundant modifier, as all interface methods are static.


maybe i was unclear with what i meant, i think that value of "K" is the result of equation 7 and 8 with kh as input (replace Y with Kh in eq 7 & 8) But which of the two alternative should you use?


what does this test test?


The images resize but it shows all the 3 images in block.


why?


Please refactor classes to packages. It's a lot of them now.


А зачем State одновременно хранится в полях, и создается заново и возвращается другими методами? Вроде то, что хранится в полях, никак не используется.


unack, needs work :)


That is just wrong.

It works here.
Did you `./configure`? It removes the line you point out on 4.02.1.


@krisan13 The 'lecture undefined' errors in the client console originated from the fact that the test data was not yet inserted before the Router already rendered the template. Thereby the lecture is not yet defined in 'isAuthor' and 'getNumberOfMembersInLecture'.

This should prohibit the "lecture" undefined errors of the tests in the client console. Please check to make sure it works on your machine as well ;)


Podes usar um método do servidor aqui para não deixar fixo o endereço que tu vai redirecionar, saca só:

```JavaScript
$(location).attr('href', '@Url.Action("JogosDisponiveis", "Relatorio")');
```


Hi, why using the default style? It's **really** ugly.


We need special code in visitToken to skip validation for Interfaces at all.


Issue CLOUDSTACK-9024 refers to this commit.
https://issues.apache.org/jira/browse/CLOUDSTACK-9024


Hi, I had filed an issue regarding LXC 1.1 in #213, I never got a worker running on any of the LXC 1.1 versions, see the error messages. To have LXC 1.0 and 1.1 working, I have submitted a patch #212.


Static?


Does this need to be added to other languages, or other languages checked to make sure they do not already define a value for 5?


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


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


Done


Code duplication with Decorators?


I used opam install merlin, and it fails.


Any reason why Flow ~3.0 should not be supported?


I don't think this is safe because the code that reads the xattr previously may not have had the change above.. which means it may look for the next xattr even though the last chunk wasn't complete.  Well.. it oculd be safe, but we need to have an incompat ondisk feature to prevent old code from getting it wrong later


Why? The `return` statement seems useless here. What am I missing?


Hello. Just saw the fork and fixed it also on the original version (+ a few other changes I did today).
The `composer create-project ...` command should now work.


@Frenchaire That's called string concatenation.


I copied this from Jetty9WebSocketHandler. I'm not sure if there is a better way to check if we are using Jetty 9.3.x


Debugging artefakt?


Good catch on the semi-colon


Yes, that seems better.
Errors now reduced to:
```
Exception KeyError: KeyError(47414985870784,) in <module 'threading' from '/opt/python/lib/python2.7/threading.py'> ignored
```


**Code Quality - Nitpick**
Check the line length on line 30, see [PSR recommendations](http://www.php-fig.org/psr/psr-2/)


probably too much complicated but something like this may simplify this:

```javascript
var binded = _.memoize(function(func){ return _.bind(func, this) });
```

then when using on and off :

```javascript
$(document).on('keydown', this, binded(this.onKeydown));
```


"klub" to nazwa po czyszczeniu; "old" - nazwy z Wiki


I know.
However, that is relatively easy enough - in theory - to deal with.
If you look at the payload it will tell you the hash of the commits and then you can ask BB for that commit and get the list of files that changed.
After that, just download them directly from the repository, using the hash (commit id).
If you use http://restbrowser.bitbucket.org/ you can test it before coding it. :)


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


Oi. Uma operação de conjunto é mais leve que duas (Max e FirstOrDefault):

```C#
public String GetNomeJogoMaisCaro()
{
    return PesquisarJogoPorNome()
      .OrderByDescending(_ => _.Preco)[0]
      .Nome;
}
```


Some semi-related background:
http://stackoverflow.com/questions/8774958/keyerror-in-module-threading-after-a-successful-py-test-run


I used opam install merlin, and it works.

Can you describe your host system? Any peculiarity (BSD/OSX?) which might help understanding the failure?
Also, can you share all logs produced by opam for merlin.


Noooo, my beautiful regex :D


brym brym


**Code Quality - Awesome**
Good attempt on use of _interfaces_.


Yes, In my previous work, we were required to do it by our legal department.


Does "lock contention" mean a deadlock? Like process A and B both need docs X and Y, but A locks X first and B locks Y first, making each wait on the other to complete, resulting in a deadlock—is that the situation you're thinking of? In practice that doesn't seem to happen just given the possible interdependency patterns of form submissions, so I don't think there's much of a worry there.


Maybe check that optimizationSolver.getImplementation() is of type  OptimizationSolverImplementationPointer and not SwigPyObject?


I don't see an @items collection getting set in your controller for show, nor would I expect to. The show page should just show one item, but honestly in this app, everything is being shown on your user profile, so you really don't even need this view.


`ketrew submit` is not power**ful** nor flexible ??

It's just a way of testing the configuration, it should not be used in real life.


self.log_handler should be removed in this logger statement.
logger.info(output)


comentário teste de fim do código


(plus `chroot()` in the setup func)

 ...although that would lose the container namespacing.  Eh.


This is not an error.  Should be logger.info.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3561](https://teamcity.mesosphere.io/viewLog.html?buildId=68064&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:58


Note: These are LaissezFaire to retain backward semantic compatibility with prior usage. They should eventually be moved to Strict/VeryStrict as default.


No. :)


Pra deixar o código mais flexível:

```C#
var brl = new CultureInfo("pt-BR").NumberFormat.CurrencySymbol;
return String.Format("{0} {1:0.00}", brl, lista.Average(_ => _.Preco));
```


Achei estranho essa parte do código.


logger.debug please


I reported the output over there https://github.com/the-lambda-church/merlin/issues/460


@romani
Got it


@rooty0 да я чет не предполагал, что кому-то пригодится, там одна публичная функция пока в общем-то https://github.com/l0kix2/python-debparse/blob/master/debparse/deb_control/__init__.py#L9 — парсит debian/control — объекты, которые получаются можно поизучать в консоли или посмотреть на их классы в коде.


Prefira `String.Empty` invés de `""`

Motivos: legibilidade e semântica.


why is this in the else? I would want to be able to combine showing bears and deleting tags, that's no exclusive IMO


Why did his page get deleted/moved?


I always use `'button[data-action="save"]` since not using `"` has several case of blowing (which happens more often with higher version of jQuery), but if it works then ok


:+1: para o correto encapsulamento


To check if version exists try in terminal
```sh
apt-cache policy sublime-text
```


This may help - I think the re-spawn event was registered wrong - can't test atm as changes made in browser git ;)


unack, needs work


Cool for stub reuse. But do you do _unit_ testing? DO _real_ functionality of DecoratorFactory called? Is it pure stub?


Hi! I've started using DVR in a new project, and in trying to integrate it with Xcode 7 via Carthage, I've come across this commit. After about 2 hours of debugging.

When first run, Xcode just crashes when it gets to my DVR enabled test. Which isn't really at all helpful. I know that technically `fatalError` would be preferred over `print` & `abort` but I think the end result is not at all helpful because now these messages and instructions don't get printed.

It's especially important, because the output path is actually buried inside the project's derived data (which might not be obvious to many DVR consumers - the default output path is `"~/Desktop/DVR"` so I was looking on my desktop for some JSON like a silly billy)

And without the helpful instructions (and Xcode stopping on the line, and not just unceremoniously crashing)  - it's a mystery what happened (because the crash log is just some inscrutable DBGLLDBSessionThread garbage). My first thought was that DVR just hadn't been updated yet to work in Xcode 7.1, or that I'd integrated it incorrectly.

So, this has taken me a little while to get to the bottom of, and eventually, undoing this commit... and then my test stops on the  `abort()` in the "success" path of the code, and the console has this...

    Test Case '-[FooProject.BarClassTests test__foo_bar]' started.
    [DVR] Recording 'BlarBlarBlar'
    [DVR] Will output to path at /Users/daniel/Library/Developer/CoreSimulator/Devices/F2BDDD81-C52A-47E7-8A8A-0686B225C626/data/Desktop/DVR/BlarBlarBlar.json.
    [DVR] Persisted cassette at /Users/daniel/Library/Developer/CoreSimulator/Devices/F2BDDD81-C52A-47E7-8A8A-0686B225C626/data/Desktop/DVR/BlarBlarBlar.json. Please add this file to your test target

So, I think while @stephencelis probably thought he was improving the code - I think it actually makes it so much worse for adopting/using DVR.

Stay tuned, will create an issue & PR to restore what I think was the intended behavior.


We have *system level* caches being defined here for example:
https://github.com/shotgunsoftware/tk-core/blob/master/python/tank/pipelineconfig_factory.py#L637

For apps etc, we also have a cache hook where a user can configure the location: https://github.com/shotgunsoftware/tk-core/blob/master/hooks/cache_location.py#L51

For system things, it's probably not good to use the hook - there are many chicken/egg cases where we need a cache but don't have a hook yet. But what we should probably do is to cetranlize some utility methods for these system level cache files, so that both the logic in the pipeline config factory and the yaml cache files can use the same method to determine the cache location.


logger.info