It is not a good idea to run  `docker-compose up &` in the background (using the '&' sign). This way Ansible cannot catch the stdout/stderr messages ! So if the command fails, ansible detects that it was successful as the returned code is always zero.


TeamCity Bender :: Deploy [Build 3.0.17.0](http://build.mikeobrien.net/viewLog.html?buildId=2450&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3243, ignored: 5 Build time: 00:01:21


I have been looking for a way to force cachecontrol to do that, but I don't think there is a method built into cachecontrol to do that.

We will figure out a clever way to handle it tonight.

It's not as bad as it sounds right now though, the code hits the api once for each show per day, unless it has a mapping, then it becomes twice for that day (because we added the http://thexem.de/map/havemap?origin=  by xem's request the other day)

Around 100 requests a day for the average user is not super ridiculous, but the number of hits definitely can be probably reduced by 90% or so.


I updated the gemfile, nokogiri is no longer pinned. I ran the tests, deployed to test and tested there. All seemed OK. If it truly does need to be pinned then lets discuss.


Are you being sarcastic?  I'm not sure what you're getting at.  Established compound words like "filename" are O.K. either way, but words that aren't commonly used together in a compound word aren't.


@j-pb I stand humbly corrected, but remain further entertained. Even the most noble of intentions can produce the most vapid and meaningless of solutions, something you yourself were talking about earlier in this thread. Case in point, the irony of a "Homelessness Awareness" section appended on the end of a hobo name generator. https://github.com/ybur-yug/hobos/issues/1 is a solution looking for a problem that doesn't exist.


merged the work in progess files by accident...


I am currently experimenting with mandelbox, where the  weight function
appears to be much more expensive than with the mandelbulb ( 4 weight
functions almost  doubled my render time).  This needs to be investigated
further and maybe " enable weight checkBox" may be required.  Weight
function often works well up to 1.3, consider allowing spinbox and/ or
slider from 0 to  2.0 range. Weight function may need to be applied to  aux
DE for spherical fold.

On Thu, Jul 30, 2015 at 8:09 AM, Sebastian Jennen <notifications@github.com>
wrote:

> yes, looks like auto generated, see hotkeys under menu > image
> -> autohotkeying tries to find first occurence of not yet used capital
> letter in any surrounding.
> Not really desired, although it could be quite handy if we got ctrl +
> ??? as hotkeys for common things like save / load / settings / ...
> You can add a shortcut as a property directly to an action in the
> designer. This will also write the hotkey next to the entry.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddhi1980/mandelbulber2/commit/c0f01842c1044417f63e5ab549a0bbb39fffe77f#commitcomment-12429963>
> .
>


Yes. You must open them in a photo editor and change the dimensions there. If you have a Mac, you could do this using the Preview app.


cool


The demo project build a 83.1 kB bundle.js file.


@krainboltgreene

Your hypocrisy is astounding. My meme image was just a slightly more blunt way of saying what
you have been saying all of us over and over. YOU are spreading toxicity.


I looked at the db and it seemed that the comment field showed whether or
not it was checked whereas the answer field was always 5 for checkbox

On Wed, Jul 29, 2015 at 8:57 PM, Shoaib Akbar <shoaib429@gmail.com> wrote:

>
> On Wed, Jul 29, 2015 at 8:28 PM, Yang Song <notifications@github.com>
> wrote:
>
>> answer = Answer.where(question_id: self.id) will return a list of answer
>> objects. It seems to me that this should be implemented like (I could be
>> wrong since I miss part of the discussion):
>>
>> def self.checked?(response_id)
>> answer = Answer.where(question_id: self.id, response_id:
>> response_id).first
>> answer.comment == '0'
>> end
>>
>> And one more question, for checkbox questions, are we going to use
>> comment field or answer field to store if it is checked or not?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/expertiza/expertiza/commit/6dc80821a403fe854c5c57b1de4474d1575c7eb8#commitcomment-12433921>
>> .
>>
>
>


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1776 of 1778 tests passed)

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


Actially, the fork was written for a battlefield plugin I'm writing. The attachments would have been equipped by the plugin and not the players sooooi


Ha! Yes, I can see that. But I don't see how that relates to using JSONSchema to validate bodies?


I take this comment back. The define is OK.


1709-ComponentSetpointScheme (kbenne) - i386-Windows-7-VisualStudio-12: OK (2200 of 2232 tests passed)

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


Did you mean to include imperial legion specific unit map entries "UberCommander", "AnyUberBasicFabber", or "AnyUberAdvancedFabber"?


Oops, my bad. This was about the above thing, not the below thing :-P

Lots of people have been asking for schema validation. It is something to use instead of V2 matching. If a schema is provided, use that instead of the matching rules, I guess. It means that if someone is already using a schema on their project, they can include that in their pact files. I guess it would be more helpful with XML than JSON.


Please use the SSpacer element with the "height" parameter to create empty lines / padding. It will be faster loading up the interface than using empty description elements.


Need to modify the naming scheme here in case of potentially overwriting operator classes in the operation_registry. Might happen when a user is trying to make overloaded functions.


I'm happy to de-scope it in favour of the other items.


I just say that I'm perfectly comfortable with having areastore.cpp, nodedef.h, minimap.cpp, nodemetadata.cpp, httpfetch.cpp, hud.cpp, etc.
after your logic, we would have to write h_u_d.


might not have a cartItem to delete if it's the case where a new row entry was created


> I would like a Process_Mark_Stack() here anyway.

The issue is that Mark_Devices loops, and in the loop it calls the top-level MARK_SERIES_DEEP which guarantees it will finish with the mark stack empty.  So in that loop if you wanted the Process_Mark_Stack() to be called explicitly, you would do PUSH_MARK_SERIES_DEEP() each time through the loop, potentially accumulating a longer list than you need to just for the purposes of being able to call Process_Mark_Stack().  This is the concept of MARK_SERIES_DEEP; to self-check that it is called at the top level only, and that it finishes with all its markings done.

So my concept here was that any time you say "Push" you are saying "defer the marking, don't do it now", but if you don't say "Push" then it's understood it does all its work before it returns.  Since it is named `Mark_Devices` and not `Push_Mark_Devices`, you are supposed to be feel confident that it finishes.

But in other cases, I've helped resolve reading ambiguity by forcing the name either end with "DEEP" or "ONLY".  So perhaps that should be done here too by making the name `Mark_Devices_Deep`.

Beyond that...there could be an ASSERT_NO_PENDING_GC_MARKS macro, and it could be called at the start and end of Recycle().  If you really wanted a call to that after Mark_Devices_Deep() it would not hurt anything, being a cheap assert.  But it's not a good idea to call Process_Mark_Stack()...an operation that can "do work"...with the appearance of re-doing work that should have already been done under the invariants in place.

*(Another possibility would be to just inline the code in place so one could see directly that it doesn't do any pushes.  I actually am not a big fan of making functions for something that is called exactly once; length of a routine does not bother me if that's how long it is.  Making a function can effectively makes the code longer, not shorter, under the "appearance" of modularity.)*


A lot of people have mentioned it to me too, and then when I've explained why we don't use it (it's the first FAQ on the wiki), I've not had anyone come back with a problem that only using the schema would solve. I'd say, lets put it at the end of the priorities. I wouldn't put the release of v3 on hold for it.


Also, I ask where do you draw the line? map_gen or mapgen? You'd like to write file_cache.cpp, but keep filesys.h?


Probably beyond the scope of this feature, but could prove useful (allow pulling of the title in the Clinical Report gui), although may be a lot of effort for small gain.


No, it's not because running an internet connected WinXP machine in 2015 is actively harmful to the network as a whole. Without security updates your system is a botnet host waiting to happen.


@USA-RedDragon np, was just pointing it out.


"verrue", monsieur!


Do we need `&& ! defined(__clang__) && ! defined( __GNUG__)` here?  I'm sure clang and gcc don't define `_MSC_VER`.


The reason why `docker-compose up` failed is that you have not copied docker-compose.yml on the remote machine. docker-compose didn't find docker-compose.yml so it raised :
"""
Can't find a suitable configuration file in this directory or any parent. Are you in the right directory?
Supported filenames: docker-compose.yml, docker-compose.yaml, fig.yml, fig.yaml
"""


Does it make sense to translate labels in the xml file (I figured it was meant to be standardized)?


Actually, is there a way to define this flag once, and not per-platform? We can add it to an implementation file that doesn't use it too. Then use `DECLARE_bool(disable_kernel)` where ever it's used.


Good choice of test here.  Nice job.


Why include the mode in the search? If you don't need the mode you can make this a lot faster with:
```cpp
results = SQL::selectAllFrom("file", "path", Tables::EQUAL, "/var/osquery/.gtfo");
```


[The documentation for RemoveFontResourceEx](https://msdn.microsoft.com/en-us/library/dd162923.aspx) suggests that it's important to call RemoveFontResourceEx the same number of times that AddFontResourceEx succeeded....so the return values should be checked?


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1775 of 1777 tests passed)

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


代码段缺少标记导致代码后的链接部分无法正常显示


Nacho pusiste que era la version 3.0 y es la 4.0


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


because #2682


Windows compatibility isn't there yet as getopt is linux only, but I'll replace it.


fast-forwarding master to borshop-integration-282-test/mra/gh280 = ca280a2a


I ran into this also. It works if you `use Python-3.4` .


@Keno should https://github.com/JuliaLang/julia/blob/a2a218bfd4f9e8d306b1f5dee7814f94d462bcb3/src/debuginfo.cpp#L329 also be ifdefed? I have to change that line as well to build with svn.


+ agreed. you just have to keep updating Grunt. precisely.


Customized error messages:
https://docs.djangoproject.com/en/1.8/topics/forms/modelforms/

testing... ... ...


quá it hàm cần làm thêm cho đúng yêu cầu


Good job!!!


weird space.


chưa set password rỗng chỗ mỗi user trong list


With regarding to the comment: REBSER is managed by GC and only the data part is not. This is only used in this case for now:
make struct! [
             [raw-memory: location]
             i [int32]
             j [int32]
]
as trying to map a C struct to a rebol struct! (in this example, C struct {int32_t i; int32_j} at memory location will be mapped to a rebol struct!. The REBSER used to store the data has a flag of REB_EXT, so that the same memory can be shared by both C and rebol, so the changes from either side will reflect on the other side.

Here is a real example: https://github.com/zsx/c2r3/blob/master/lib/c2r3.reb#L510


Great work! That was quick!


Just a side cheer: These two are equivalent if both \lim a_n and \lim b_n are finite.


chưa set mật khẩu rỗng cho user trả về


ssssssssssssssss


Since this is specific to bootstrap base, can we rename it to generate-bootstrap-base-child.


Much better to use the getListItemTitle() function to collect a lits item title which will deal with translation if needed. https://github.com/openemr/openemr/blob/master/library/options.inc.php#L2968
(also, out of curiosity to you mean to have the value be the title and not the option_id; just curious)


createBootstrapBaseChild


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


bootrstrapBaseChild


@AlexandreDC veja isso


Agreed. I'm still debugging exactly why my git failed when 2.5 on your end does not. I also have several other test failures I am running into, and I think they're also related.

Somehow the CLI implementation finds the "origin/master^" as a filename, and fails. I tested with a specialized clean configuration by faking $HOME on the environment to mvn.

However, considering that my environment clearly works for the git commands, I'd like to determine and debug where the configuration changes break tests and try to make the tests (or the CLI implementation itself!) more robust to to configuration changes.

In addition, I faked the jenkins version so that the tests would stop failing on Java8, since I have no success using Java7, and can't even find a way to download it from the official oracle site.

I believe these should be valid test results, as the only change is the Jenkins version and to 1.560 and not any other dependency.


/cc @jridgewell & https://github.com/jashkenas/underscore/pull/2231


- [ ] this is an incomplete item


technically one could be running ReactOS, perhaps they support vista API though.


I haven't removed it's use just being more picky about it.


This makes me really sad. The original implementation made a bunch of design choices that doesn't feel fantastic at all. The fact that FragmentSize is exported at all is kinda ugh.


Never heard of that OS, interesting.


không được return về null, đồng thời check null đối số
```
if(user == null){
   throw new IllegalArgumentException("user could not empty");
}
User userCheck = getUserByEmail(user.getEmail());
if(userCheck != null && StringUtils.IsNotBlank(userCheck.getId()){
throw new RuntimeException("");
}
```


Why use json loader?


this should be:
``````
$optionsStr .= "<option value='".attr($org_row['id'])."' $selected>".  text($lab_org_name)."</option>";


```
/home/peter/TrinityCore/src/server/game/Handlers/CalendarHandler.cpp:336:64: warning: implicit conversion of NULL constant to 'uint64' (aka 'unsigned long') [-Wnull-conversion]
        CalendarInvite invite(sCalendarMgr->GetFreeInviteId(), NULL, inviteeGuid, playerGuid, CALENDAR_DEFAULT_RESPONSE_TIME, CALENDAR_STATUS_INVITED, CALENDAR_RANK_PLAYER, "");
                       ~~~~~~                                  ^~~~


I hope you are doing well!


probably, I just noticed the one and figured I'd put in on master.


Not even attributing me as the commit author? :stuck_out_tongue_winking_eye:

Kidding, of course. I don't really care.


I should not have let this one sneak by me.


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2217 of 2232 tests passed)

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


Haha. That commit is like 3 months old. A few people plusoned on this one if I remember correctly. What is this sudden change of mind?


And to change the actual base layer, I uploaded our EPSG:3338 Natural Earth GeoTIFF to my local GeoNode instance, then created a `geonode/geonode/local_settings.py` file with the following contents:

```
MAP_BASELAYERS = [{
    "source": {"ptype": "gxp_wmscsource", "restUrl": "/gs/rest", "url": "http://localhost:8080/geoserver/wms"},
    "name": "transparent_clipped_ne",
    "group": "background",
    "visibility": True
}]
```


@danielmarbach  How about the callback subqueue functionality, shouldn't that also be moved out ?


r+


well either way `SynchronousOptions` is wrong for usage on a async api


@AndrePaiva você testou esse código?
Acredito que ele não vai retornar nada...
Alguma sugestão a mais?


Here is something you can do to add to your shell script skills. I learned shell scripting while interning at Ft. Benning's Maneuver Center of Excellence Battle Lab. What I did was create a factorial program in Java. I would basically make a for loop  from 1 to 31 and send it to the java program. Then take that output and store that in a file. Basically use ">>" to update a file without rewriting it. Using > will overwrite the file.
Using this method is a great way to test out programs.


Se introdujo un erro de compilación en PushManager y CommitManager.  Fue arreglado en el siguiente commit.


@myshkin5  We should have split the logging refactor / cleanup from this bug fix.   Mixing them in the same commit clutters things.


Try to make Matlab plot multiple figures in one launch.
Try to use customized icon in Matlab plot function


Acho que só isso não resolve....
e se eu digitar: INCoNSTITUCIoNaLISSIMaMENTE?


Good point. I remember someone saying this one is done. Just merge and doco ;)


Should changing "4" -> sizeof(REBCNT) be a separate commit?


saw approval from matschaffer
at https://github.com/graydon/stellar-core/commit/4c21eb2f073306c425c5b5f2c7da8baf90663b26


merging graydon/stellar-core/switch-testnet-keys = 4c21eb2f into auto


graydon/stellar-core/switch-testnet-keys = 4c21eb2f merged ok, testing candidate = cd572bc6


Fala @leandromac!
Adicionei esse cara para gerenciar os perfis do nosso usuário, vc pode dar uma olhada no comportamento desse método aqui http://devdocs.io/rails/activerecord/enum

Também tive que criar uma migração(https://github.com/leandromac/ticket/commit/d4daae1e9b79bec26225ed87948666fad27a1c56#diff-799a252233860fb8795eb8553b3a630b) para que o nosso campo `role` no banco de dados seja um inteiro.


No, here's a preview.

<img width="856" alt="screen shot 2015-07-29 at 21 49 29" src="https://cloud.githubusercontent.com/assets/1497697/8974085/ccc7131e-363b-11e5-86ef-7710bd59cf41.png">


It is using now SendOptions


Sorry @danielmarbach been busy with SI/SC, so I probably missed this change.
Anyway, I thought it would be better to express my opinion now then never.
Also I am referring to `RequestWithTransientlyHandledResponseAsync` I think is now called `RequestWithTransientlyHandledResponse`


I wonder if some people will confuse this with Italics. There are some flavours of Markdown that use the __bold__ and _italics_ underscores.


@myshkin5 Moving the select on the timer into the !connected block will break with the http syslog sink... this sync doesn't have the concept of "connected" as there is no state in the http connection.   The result of this change will mean an http endpoint will never backoff.


Sorry @danielmarbach, I was referring to the original PR, moving the feature out, is a bit more work


If you really need to work with Java 8, maybe you should instead merge in a copy of my [proto-jdk8 branch](https://github.com/MarkEWaite/git-client-plugin/tree/proto-jdk8).  I believe it still compiles cleanly on Java 8, and depends on a Jenkins core version which works with Java 8.

That can't be included in the pull request, but it would allow you to avoid the ugly work arounds for Java 8.


Typo here: `public`


I created a pull request and merged it in


I'll fix this when I merge.


Yes, I agree that's important, although I doubt anyone will think deeply enough to care.

I was also independently thinking of a rule that we wait for a minimum of X seconds, then either for a Y% or maximum of Z seconds, for which we'd start the game. Our first days can guide exactly what those values are. I don't think we need to hardcode them, but they'll be useful guidelines and we can bring a stopwatch for timing purposes.


Not quite clear what the identifying series is referring to. Could you be more clear, or give some examples? ideally an assertion?


I believe the semicolon shouldn't be here, as the -i option doesn't get any arguments. Right now, if I use:
`./arduino-serial -p /dev/ttyACM1 -i`, the command fails with `./arduino-serial: option requires an argument -- 'i'` error.
Current workaround is to add a dummy argument after `-i` option, but it's ugly.


@PhilJay, I have another change in `calcMinMax` in BarChartView, but I don't see a similar change in android? I simply change `maxEntry` to `barData.xValCount` in https://github.com/danielgindi/ios-charts/pull/248/files#r35759740


We should not conflict with livestreams, even if they are set to unimportant (=50)
https://github.com/tvheadend/tvheadend/blob/master/src/profile.c#L850


In my opinion all actions from here to line 68 should be hidden into a wrapper class, that will serve as a data model. I prefer our controllers as thin as possible. It will also help us in the future with nested deployment execution task.
Something like:
``` Ruby
stack = StackDefinition.read_yaml(stack)
stack.schedule!
```
It will also enable us to create good visualization for the stack. For visualization sake I want to be able to understand the status of each node in the tasks graph.


This could be a `p` element (as you suggested).


@KrauseFx  I see that this class contains both device_type and language which are not part of the original data structure returned in the JSONs. Is this necessary ? Maybe for the deliver integration ?

https://github.com/KrauseFx/deliver/commit/6c1189e01b08ae6c3dd761dbe86c3843e52a1075

I would like to reuse the same structure for all the other uploads, and I wonder if we can find a different way of keeping that mapping to keep the app_screenshot class closer to what ITC uses, and maybe make it a Base data structure itself instead. Or maybe I'll introduce a parent class instead.


# 作出修改
改成hello world


Do we really need an element around this?


Done i think.


Updated 1ee370e934b8b364c2b9d32b718f366788085986.


Oh, because `None`/`nil` isn't ever going to be set, right. Will do.


@ogoffart does this make the second backup be `ownCloud (backup 1)` or `ownCloud (backup 2)`? Should be 2, since backup 1 is already the one without the number. :)


Then I'll change all custom SQL queries made by previous developer to use wp_query.


Merci! This is definitive an improvement, though I do not know if we should be analyzing the file at hand for the type of line endings. The latter approach would enable an OSX build of the postviz to parse a simulation computed on a windos machine.


Oh noes... I got wikipediad :(


nothing personel kid


The method must be annotated with @Override to avoid WriteOnReadError here


Actually, the code I copied over from RSpec core is from RSpec 3.2 – so yeah, should be fully compatible :)


@KoFish blocking/removing/changing content due to a content policy _is_ censorship.


I think it would be nicer to use factory method pattern here:
``` Ruby
class Validator
  def self.validate!(deployment)
    v = Validator.new
    v.validate!
  end
#...
end
```
and then in the code just call `Validator.validate(deployment)`


tiplayer style change


Please change this to non-greedy option instead `/\[(.*?)\]$/` then it will be less greedy.


Why the `hasattr`? Does `dir` return something for which `hasattr` doesn't hold?


I see. I will update the release notes.


Yes, this is a work in progress. This documentation is wrong, but we still don't know for sure how to package mongo-logs.

Thanks for the reminder @Simage ;)


Ack to the general design but I'd prefer it if we did the 'grep' in python; there's really no need to fire up a full-grown FSM regex engine to go looking for one little string in a tiny file.


debug


This needs to be `hg+https://...`. I'm sorry that Jeremy was confused and initially told you `hg://https://` instead.


Fix #103


write somthing meaningful


You all lose. Just leave. Go form your own git hosting service that'll wither and die. Shoo.


If you'd like to generalize that even better - say add a `is_module_loaded()` method to the base Plugin class then we can just have:

```python
        if self.is_module_loaded("bridge"):
            brctl_file = self.get_cmd_output_now("brctl show")
            if brctl_file:
                for br_name in self.get_bridge_name(brctl_file):
                    self.add_cmd_output([
                            "brctl showstp "+br_name,
                            "brctl showmacs "+br_name
                    ])
```


Thanks it was an accident :$


https://yourlogicalfallacyis.com/ad-hominem

(logic is a pretty popular device in programming, btw)


Couldn't this file be moved to files/samples-contrib , since it's marked as notofficial?


A okay. I see.

Even if you've not introduced that code, may I kindly ask you to add constants with meaningful names in pvr.hts for the priorities defined in tvheadend? Those magic numbers are never good. ;-)


Please put your mail address here


Fix #104


This needs to either point to an installation script (very discouraged) or be changed to a platform-specific package name. If *this* version of the bindings works with the present `libtidy5-dev` then that is what should be used. It is unlikely that the package name will change when it is updated in the OS repositories so using that name will still be safe. The opam depext mechanism presently has no facility to express external dependency version constraints. You can give some guidance to your users with `post-messages` like: <https://github.com/ocaml/opam-repository/blob/master/packages/sodium/sodium.0.5.0/opam#L34-39>.


> Couldn't this file be moved to files/samples-contrib , since it's marked
> as notofficial?

It is actually official now. I should have droped the part "notofficial"
too. Will do this.

Bye,
Chris.


Still need to fix initial directory to current site


What's the point of this test?

The benefit of a test is correlated with the probability that the test catches a bug in the implementation, and inversely correlated with my confidence that the tested code works (without testing it). (That's why simpler code is better - I can be more confident that it works correctly if it's easier to reason about it.)

The cost of a test is that it has to be maintained, adds running time to the test suite, and may be buggy itself.

In this case the test code duplicates the function's implementation. That reduces the benefit to zero - if I have a bug in the implementation, it will be in the test as well.


@lox are you using `docker-compose` in this image in prod? Because from my testing (and reports) it doesn't work (because glibc). The only way to install is via python and pip (ala https://github.com/blueimp/dind/blob/master/Dockerfile). I'll update them, I just wanted to check first because I swore you'd used this prod already.


什么鬼。。。。。好无聊的样子(+﹏+)~狂晕


This is what hybrid mode is for ;)


You left debug code here, maybe unintentionally.


Great!


@krainboltgreene Thanks for making this environment hostile!
@vibhavp @Yuisbean @KoFish I'm not sure if you can exactly call this censorship since retard is used here as an (ironic) insultment and insultments doesn't necessarily get protected by the right for freedom of speech.


no, the fix was applied on the master (dev) branch, but the online documentation is the one from the last 2.1.3 stable version (where the issue was not present).

if you need the documentation fro the master branch you can build it using `grunt doc`


The creation of the execution degree site's internal pages is throwing an NPE if a proper theme was not uploaded. Since this code will be triggered in the creation of execution courses (through signals), an execution course cannot be created unless a proper theme exists. I believe you should avoid this error or throw a proper error message.


Thank you !


what is this phani


Option string has a typo, it should be: JEV_INSTAL**L**_MANAGE


I have created a pull request for the proposed skipping of the rotation matrix reset in G28.
"G28 S" skips the reset of the rotation matrix. It seems to work for me but need to test it with some printing in the weekend. (I left the Delta code out for the moment since I can't test that)

  // For auto bed leveling, clear the level matrix
  bool skipclear = code_seen('S');
  #ifdef ENABLE_AUTO_BED_LEVELING
                if (skipclear = false) {
             plan_bed_level_matrix.set_to_identity(); //if G28 S then keep the rotation matrix else clear
                }
    #ifdef DELTA
      reset_bed_level();
    #endif
  #endif


Hmmmm, It would take me a while to a adapt that, but it seems like a fair point you had..


@Jeavon  There is no null check here, this causes YSODs and the query builder to break if targetNode is null


add me as a contributer. im too dumb to delete ur stuff so u dont have to worry


Que show Danilo.
Vou da uma conferida.

Valeu ;)

Em 29 de julho de 2015 22:50, Danilo Resende <notifications@github.com>
escreveu:

> Fala @leandromac <https://github.com/leandromac>!
> Adicionei esse cara para gerenciar os perfis do nosso usuário, vc pode dar
> uma olhada no comportamento desse método aqui
> http://devdocs.io/rails/activerecord/enum
>
> Também tive que criar uma migração(d4daae1
> #diff-799a252233860fb8795eb8553b3a630b
> <https://github.com/leandromac/ticket/commit/d4daae1e9b79bec26225ed87948666fad27a1c56#diff-799a252233860fb8795eb8553b3a630b>)
> para que o nosso campo role no banco de dados seja um inteiro.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/leandromac/ticket/commit/d4daae1e9b79bec26225ed87948666fad27a1c56#commitcomment-12434847>
> .
>


I tested the cmake branch out and it works great :)


The cmake branch works great!


> Reply to this email directly or view it on GitHub
> <https://github.com/alpinebits/developers-kit/commit/db0da2eba286535f7bc9dc9306cd82ff4c81ab0f#commitcomment-12440298>.

Wait.

I already dropped the non official part.

This is correct as is.

All ist good :)

Bye,
Chris.


去除冲突


good change


camelCase


@Shazwazza Wow, it seems like an age ago that we wrote this! I'm currently on vacation so can you add a null check?


use generated serial version!


> Great!

It was already official with 2014-04:

In case of success, the OTA_ResRetrieveRS response will contain a
single, empty Success element followed by a ReservationsList element
with zero or more HotelReservation elements containing the requested
information (zero elements indicate the server has no information for
the client at this point).

It's just that I was too lazy to include it in the samples but kept
it with the stupid name for myself. Then it somehow was released and
with this name change its just part of the official samples and
they lived happy thereafter and had many children.

Bye,
Chris.


wtf?


These changes (especially the .find calls) need jQuery. See my amends inline.


What if db_out is not None? We would skip that part.. (that was my concern, but I guess inst_out is None if and only iff db_out is None, so it is safe). Will change it.


Use patch method for partial updates of a resource. Add data-confirm attribute to trigger a model dialog.


You should use $scope.grid.renderContainers.body.headerCanvas instead.


Replace default browser model dialog with a jQuery model dialog.


Read next comment ;-)


已提交修复


You should use gridUtil.closestElm(movingElm, '.ui-grid-render-container').querySelectorAll('.ui-grid-viewport') instead.


remove unused code


You should use movingElm[0].offsetParent instead.


fix encoding!


Allowing render_views to be true for all tests interferes with some tests. In particular, it interferes with settings#index since the current_user is not set in the view. Will selectively turn on rendering of views for now.


@ellbee fixed, thanks. Misunderstood that


Need to turn on render_views to allow jbuilder views to render. Otherwise, the HTTP response body is empty, i.e., no JSON object.


when mongo server is not available, routine will log this message instead of die the process.
also add a function that can detect if connection was build successfully.


Use a partial view to render nested links JSON objects.


I hadn't come across the [double splat][] before so was interested to learn about it. @floehopper and I discussed its use here and noted that the current implementation is equivalent to `def initialize(args = {})`.

[double splat]: http://stackoverflow.com/questions/18289152/what-does-a-double-splat-operator-do


The same as with `Validator`, probably can be wrapped in a factory method


@patriknw what do you think about this CRDT as an example for the book?


Wait, scratch that....

The whole point of this change is to support tornado's async http, which requires the patching to be in place even when we exit the stackframe of the original generator/coroutine.

I had some fun writing a test for the behavior you described before this thought hit me, and if you really feel strongly about it, you could put this functionality behind a flag if you want. Still, I think the more useful behavior is to have the cassette be active while the generator is active (unless explicitly replaced).

@kevin1024 @abhinav thoughts?

Here is my test for the behavior @graingert is describing

https://gist.github.com/d4bfeaed2a22d0d5908f


According to [this](https://github.com/alphagov/smart-answers/blob/8077b6bdff9281aac6294f656d5fa5a237cb5855/lib/smart_answer_flows/README.md#defining-next-node-rules) `next_node { }` syntax is deprecated, because it removes info from the `/visualise` tool.


I couldn't immediately see how `#floor` was taking an argument. @floehopper explained that the [`BigDecimal#floor` method](http://ruby-doc.org/stdlib-1.9.3/libdoc/bigdecimal/rdoc/BigDecimal.html#method-i-floor) takes the number of decimal places to include in the result.


Is there a better way to add this colour here and in line 306?


The wording is probably not the best. We have some other infrastructures that we are testing out where I don't think this is available either. I'm not sure what databases are preinstalled on OS X, what I was referring to being unavailable is the `services` configuration, which currently only works on Linux and not on OS X.


I thought `^` was some kind of magic `Hash` combination method but @floehopper pointed out that it's xor and that this `hash` method is returning a hash code and not a `Hash` object.


just to let you know. for me, this commit created a problem.
i see that i am always at 230mb free ram from 2GB. and nothing changed in my apps in RAM.
device was always in OOM state, LMK was kicking apps from RAM every minute.

When reverted, all back to normal. free ram above 400mb and all my stuff in RAM as was before.
Check if you see such issue. it's takes time for this to happen. may take 3 to 8 hours. then only reboot help.


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1776 of 1777 tests passed)

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


It took me a while to realise that it includes a leap day because 2016 is a leap year. @floehopper and I discussed making this clear in the test name somehow.


@neappx There's more than one definition of 'censorship'. In the one used here (preventing speech of any kind because of personal disagreement or dislike), 'the right for freedom of speech' (which varies heavily by jurisdiction) is not a relevant factor, and this absolutely *is* censorship.

Whether that censorship is *justifiable*, is a separate discussion.

I do not believe that it is, for the simple reason that essentially 'taking over' and monopolizing an entire ecosystem (developers, in this case) comes with a social responsibility to provide a neutral platform, so as to prevent exclusion from that ecosystem. What the ToS or content policy say is not relevant there.


@neappx Freedom of Speech is a constitutional right which only says that the _Government_ cannot prosecute you for what you speak. This is completely different.


This changes breaks quite a few CORS requests to servers that do not accept this additional header (like the google maps API). And even for those that allow the extra header, it causes an additional preflight request.


@floehopper and I talked about the use of these "magic" numbers in tests. James explained that they come from external test data. I wondered whether we could duplicate the calculation in the test rather than rely on the number alone. My concern is that if this test starts failing in future because the number has changed that the easiest thing will be to change the number rather than to understand the failure.


please rename `apt-dependencies.txt` to `dev-dependencies.txt`


keep only `wget` here and move the others into `dev-dependencies`


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Ah...excellent idea. Thanks, octocat!


I think these are required only for `3.7`. Nevertheless, pip can install packages from a requirements file.


On 07/29/15 16:39, Pierre wrote:
> Oui tu as raison, je sais pas pourquoi je pensais que le ROLE_ était
> juste ajouté côté client par le javascript, mais pas du tout en fait. Il
> y a le ROLE_ dans le header.
>
> Je modifie ça du coup

J'avoue que c'est pas super clair pour le end-user/admin parce que
ldapadmin cache ROLE_ partout, mais dans le ldap et les headers c'est
bien ROLE_.. ca vient de geoserver a la base ca. donc imo c'est mieux
d'etre explicite :)

--
Landry Breuil
Mouton a 5 pattes du CRAIG


It would also add the cluster to the help message.


Thanks, I will test and check. :+1:


so the remainder is actually the cluster name here?


Removed part of the consensus sequence not covered by the centroid sequence. In the MSA, these positions are replaced by plus signs.


For compatibility with usearch.


名前を見たたけでどこから遷移したか、何のフラグかなどが分かるようにしたほうが良いと思います。


DRY :-)


@vibhavp
> GitHub has some (arguably stupid) content policy which doesn't allow the word "retard" in its repositories

Well, that is the worst part - it doesn't. There is no list of banned words, and there was no indication, that word "retard" is *wrong*. This means, that GitHub can delete pretty much anything (and also all forks), just because some retard from techsupport thinks ther *may* be *wrong* word somewhere.


Extraneous quote.


Missing trailing quote. Same below.


Laci and me are not sure, that the packaged incubator supports grok (it's compiled without it). Could you check it please?


I like this error message  :)


@Bystroushaak That is unfortunately a common thing in Terms of Service for pretty much any service. It's also an often ridiculed thing - "that's just a catch-all clause to deal with unexpected problems, they won't bother you with it."

And well, this is what that can turn into...


please rename `incubator-source` to `source`


This is also why I have big problem with DIY censorship and all this shit like CoC - it gives big power to hands of individuals who don't give a shit about you, or your projects. There will be no trial, no court session. Don't like it? Too bad for you.


Explain Line 288 - 291?

What is the purpose of that?


Is it required to set `JAVA_HOME` in our case?


there are two additional ...


(Let's comment on the big diff of the PR instead. Can't respond without going into the commit otherwise.)

Yeah, I was unsure about the replication of the code there, but it's not entirely trivial and I want to make sure `validate_configuration()` is called correctly. It also makes sure `validate_settings()` is called at least once in tests.

Future changes to `validate_settings()` could break this test and in that case it's worth a second look at the change, I think the test is justified.

With currently 7 tests the question of test running time and maintenance is a non-issue. Let's err on the side of having a few tests too many instead of missing them.


@craig-riecke @jnfoster

We should discuss if we want to use this or not.

I've used the ppx_deriving extension (supported by OCaml 4.02, which we all use) to auto-generate pretty printers for the entire OpenFlow 1.3 library by simply adding `[@@deriving show` at the end of every type. For most datatypes, this does exactly the right thing. There are a few cases where we may want more control (e.g., printing 32-bit integers that represent IP addresses as dotted-decimal). We can do this easily by specifying a custom printer for an individual field in a datatype.

Here is the caveat: it seems as though we can't use a Camlp4 extension and a PPX extension within the same file (camlp4 appears to choke if you use a PPX extension). In principle this is not a big deal. We use these extensions right now in Frenetic:

- `cstruct.syntax` in the binary parser
- ULex and some stuff for mucking with OCaml ASTs in Frenetic_NetKAT_Parser and _Lexer
- `sexp.syntax` in a couple of type definitions as a cheap pretty printer (and we could replace these with `[@@deriving show]` and get much nicer printing)

Unfortunately, `_oasis` causes trouble. We configure our project to use syntax extensions by adding `sexplib.syntax`, `cstruct.syntax`, etc. to the `BuildDepends` section of the library. This runs Camlp4 on all files, including any file that uses a PPX extension, and the build breaks.

The solution is to remove the Camlp4 dependencies from the `_oasis` file and add them explicitly to the `_tags` file after the `OASIS_STOP` section. In this commit, I've added this line to _tags:

    <lib/*.ml{,i}> and not <lib/Frenetic_OpenFlow0x04.ml>: syntax_camlp4o, package(sexplib.syntax), package(cstruct.syntax), package(fieldslib.syntax), package(camlp4.lib), package(camlp4.extend), package(camlp4)

This line invokes Camlp4 in all files except the one where I've used the PPX extension. It's sort of ugly.

Thoughts?


This [fixes](https://github.com/rails/rails/issues/10655#issuecomment-18094629) #153 cc @gama


Yes,works fine now.Thank you very much!


My mistake, I'll repeat these comments on the PR diff.


All g


I talked with Laci about this and we decided that putting a file (`openjdk-libjvm.conf`) under `/etc/ld.so.conf.d/` would be superior.


@Bystroushaak Well, wouldn't that imply that they have a content policy which is not public?

```#endif DEVILSADVOCATE```


please move apt-get install into the same line as update (you used this convention before)


Is it the default TLS port?


possible hypocentral positions. The definition of the hypocentral position is defined in relative terms using as a reference the upper left corner of the fault surface.


Why the switch?


I personally feel that a major, monopolized platform like GitHub shouldn't have a content policy *at all*, beyond legal compliance that is required for continued operation. It's too big and diverse to be enforcing personal standards.


Where does this namespace come from?


File size.


This change is not grammatically correct; there is an abrupt change from simple present to present continuous tense in the second clause.


kiểm tra Parameter user
```
if(user == null){
    throw new IllegalArgumentException("user could not emtpy");
}
```


stick a return nil and unindent the else.


@adamralph We have R# annotations added as file already (in NSB.Core\Properties).


What if pg_port is string with space? It would be safer if e.g. argument parser would validate input as a number.


Probably best to check for r.StatusCode < 200 || r.StatusCode >= 300, as any 2xx code means success.


Aha, thanks!


서버 설명 쉽게해주셔서 감사합니다 ^_ ^


Well spotted. :+1:


Its possible for an error to have an app but the app not to have a request.

```
 my $env = $self->has_app && $self->app->has_request && $self->app->request->env;
```
will cover that case.


First line of this commit is too long :cry: Maybe, in future, write the screen-reader explanation as the commit message body?


Missing semicolon on `&raquo` here. Safari doesn't seem to mind and renders it anyway, not sure about other browsers?


sử dụng @Transactional cho service


@joepie91 @vibhavp Fair points. I also think that GitHub's action against this repo was bad, but I don't actually think that GitHub should allow everything on here since their moderation policy represents them (and maybe also the companies using it). GitHub is the current to-go platform for open source software, but I wouldn't exactly call it a monopolized platform, so I think that if they choose to censor, they should do it in a reasonable way (only ban harsh & plain hate speech e.g.) and not pull stunts like this.


This example use the sensorflare.h library to observe variables from sensorflare platform and control a PWM output.


1. ctl and mis->file point to the same socket fd. we should dup(qemu_get_fd(mis->file)) before open a write channel.
2. mis->colo_buffer may leak if we don't do failover even though qemu will exit soon
3. at this moment(ctl is closed) ,the file descriptor of ctl will assign to next open(), in this case, failover will shutdown the unexpected fd , something wrong will occur later.


LGTM


KillMode could be a link to http://www.freedesktop.org/software/systemd/man/systemd.kill.html#KillMode=


Not that I notice or measure. The difference in idle states is clearly measurable though.


This could wait forever, right?


this part could be squashed out


@neappx Honestly, I still have some faith in GitHub. As @Bystroushaak says, it might be the work of a single tech support person, and not a decision taking by the whole of GitHub


:+1: Thanks heaps for this :)


바보야


암준혁!


Ta! I've fixed it


It's LONG here, not BOOLEAN as the message says.


this check *must* be under qemu_mutex_lock_iothread(), since qemu_mutex_lock_iothread may sleep, if failover occurs at this moment,  you code will stop the vm force even though secondary had taken over finish.


I've run into this by accident. Please ignore this comment if it is completely irrelevant. It may be just me not being too familiar with way of doing things in Python world.

Are bots intended to be run using Python 2.7, Python 3 or both? Divison operator does integer division in Python 2.7 (if both dividend and divisor are integers) whereas it always float division in Python 3. That makes the above line to print zeroes as those percent values in parentheses when run using python 2.7 (default python on my machine). Changing `pages_ok` to `float(pages_ok)` in the first division, and same for `num_errors` in the other one fixed it for me, and this should probably not break anything in Python 3. Should we care about it? Or are we assuming bots are always run using python 3?


This excludes Mac users.


This needs correction too.


Students/Ghiurau A. Alexandra Cristina/Assignment 2.2/src/MainClass.java

still, what is this!? :)) why not a `for` loop!?


암준혁 바보


In master now, I see.


primay may also have 1 & 3 issue


không return null
```
User userCheck =  userDao.findByEmail(user.getEmail());
if(userCheck != null && Collectionutils.isnotempty(userCheck.getId()){
     throw new RuntimeException("duplicate user");
}
return userDao.save(user);
```


I'm not a fan of the way `$message` is passed to `full_message()` without being used and then onto `backtrace()`. Looking at this and its parent commit together we can clean this up.

`default_error_page()` only modifies the message if `show_errors` is true; in which case `full_message()` and `backtrace()` are also called.  So, I'd just construct the message here as:
```
my $message = $self->message || 'Wooops, something went wrong';
```
and use a conditional when building `%opts`:
```
$opts = { content => $self->show_errors ? $self->full_message : $message, ... }
```

Then undo the changes to `backtrace()`, or clean up the logic there if you prefer. This is the only place we need to be combining the error message and/or exception when `show_errors` is true.


Cool, now I can repro this: same issue on Windows

![](http://i.imgur.com/L4f8bfd.png)


@krainboltgreene, I'm not sure if you've noticed, but a lot of people seem to be disagreeing with you. I do not know if you are a troll, but from a previously posted image it appears you are what we'd call a "social justice warrior" or SJW for short.
Having looked at the communities of SJWs, it has always struck me as these people who are remarkably ignorant of their own ignorance. The kind of people that proclaim enlightenment whilst accusing others of ignorance. The kind who imply that "Because X, therefore X is bad", thus committing the is-ought fallacy.
I find it is often these types that get offended by the most minuscule of dissent or objection. You do not appear to fall outside this category. I do not understand what you are trying to accomplish. The communities I have observed being taken over by SJWs have all become dull, angry, `edgy`, and rather hate-filled. Often fueled by emotions whilst allowing those emotions to extend and override rationality.

Now, on the issue of github: yes, github has the legal right to do this (afaik). However, some of us users may dislike this. There are several options, complaining is one of them, and it is exactly what has happened ITT. The "don't like it here, leave" rhetoric is rather void and unbased because we - the users - are part of this environment. It's as if we should never terraform another planet because we can't live in the hostile atmosphere: it comes down to "Why should we leave instead of complain / force change?" After all, leaving may incur a larger cost to us than github's management changing its policy.


I've also used a for loop, it's commented :)) I will change this as well.


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


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


Uh... yes, thanks!
What do you think about this?

```c#
(((value << 16) & CHCOUNT_RANGE) | es)
```

```EVENTSTATE_RANGE``` is the wrong name for ```CardChangeEventCnt```.
(Probably misleading if someone tries to read & understand the code)


You're quite right. There was some work started to make visualisations work with `next_node {}` but it was never finished. I've [added a card to Trello][trello-card] to remind us to finish that work.

[trello-card]: https://trello.com/c/VPA1rG0e/40-visualise-should-take-permitted-next-nodes-into-account


bỏ TODO trong tất cả các hàm


Additionally has an extra method 'printComparisonPlots' to loop over all distinct combinations of density metric pairings, and plots all subsequent correlation plots (calling method 'compareDamageRanks' multiple times)


not sure if that's enough


Gaps in the MSA will now count equally as ordinary symbols in the computation of gaps. Example: A column containing one A, one C, one G and 2 gap symbols earlier resulted in an A in the consensus because there were fewer gap symbols (2) than ordinary symbols (3). Now the consensus will be a gap, because the gap is the most common symbol. This is for compatibility.


No worries man - hope the comments helped explain a little more about the Python language (specifically the flow-control aspect)!


Thank you, swmitra. It is working fine now. You did a great job. Its very helping Tool.


@tph-thuering better to keep generated files out of the source directory if possible — in theory it can be in a read-only folder.


Would be better. Send me to qwatayean[at]gmail.com


Maybe an `unless` would be clearer here?


I think its OK but it should not be in the notifications branch. Do one thing in each branch.


If a node with the same UUID connects it should be an error and the client node should exit(1).
This commit also will have huge conflict with 8312133378e1b19cca5a8866b5ad1e8b3ed9640b. Thats why its important to always do one thing in each branch and base that new branch on master.


This patch should really also update https://github.com/ARM-software/arm-trusted-firmware/blob/master/docs/cpu-specific-build-macros.md


It's not important but you _might_ be able to add a minus sign to the line that renders the 'passport_adviceline_contacts' partial (ie.  `<%= render partial: 'passport_adviceline_contacts.govspeak.erb' -%>`) in order to avoid the additional whitespace in the artefacts.


@neappx GitHub is absolutely a monopolized platform, thanks to the [Network Effect](https://en.wikipedia.org/wiki/Network_effect). If you wish to attract collaborators, then GitHub is effectively the only viable option - and given its complete lack of federation (which, ironically, was one of the core design goals of Git itself...) this is unlikely to change any time soon. That GitHub itself is not open-source makes things worse.

The possibility to use another platform is a merely theoretical possibility. People have found time and time again that it doesn't work in practice.

And I reiterate - I do not believe that GitHub should be censoring *anything* beyond what is legally required. Their commercial reputation is not relevant to that. They shouldn't have attempted to monopolize the developer ecosystem, if they didn't want the social responsibility that comes along with it. That attempt to monopolize is further evidenced by the fact that their own platform remains proprietary, and a few of their marketing approaches.


lgtm, thanks!


@azr You probably already store a context that you created with `cloud.WithContext` somewhere globally, right? Mostly likely in `init()`

If so, replace that code with `datastore.NewClient`. Then store the client instead of the cloud context. It's effectively the same, just not slightly abusing `Context` to hold references to things like HTTP clients and network connections.


Thanks


Similar to before (and still not important), you might be able to use `<%= render partial: ... -%>` to avoid the additional whitespace.


No, I don't have any other option in mind.


nit: The variant/revision extraction code and this isb could be compiled out when there are no errata workarounds to apply. However, I can see this code has the same pattern as the A57 errata workaround code so we can fix this in both places later.


The license picture should be GPL instead of MIT.


$produkt['stopy'][] = $track;

Ten cyklus neni potreba

:cat2:


Do you want to completely get rid of the state?
Yes, it is theoretically possible, but why?
There would be long chains of checkings and guessing, what type it is (ErlangExportFunction, ErlangFunctionCallExpression, ErlangCallbackSpec, ErlangFunctionWithArity and many others) and how to get name and arity from it.

Yes, there is no need in myArity field. But may be we leave myName and myArguments?


As of JSCS 2.0 I was under the impression ES2015 support had been improved. See https://github.com/jscs-dev/node-jscs/releases/. I've been using 2.0 with arrows and template strings and it seems to be catching issues with those just fine. Is there something still missing that would cause us to switch to ESLint here?


@HectorPerez if you add in request https://github.com/arunagw/omniauth-twitter/issues/95


I changed this commit a bit manually, because arm64 is always API 21 or higher, so the old chrome could be dropped. Changing the 'add' into a rename from git's perspective (saves diskspace).


Oprava ty zavorky[] tam neoatri


You'll want to `go get -d` instead of just `go get`, but also you have the workings backwards -- you want contiv/netplugin to be go gettable and then add your personal remote.

The problem with the other way around is that when working on netplugin itself, the imports are messed up because the *path* is incorrect due to it being from a separate fork. This method ensures that the path is always correct, and that you can work in both branches.


@joepie91 I think we need decentralized p2p github (not just git, but graphical / social mumbo jumbo around it). Maybe ethererum will be good platform to build something like that.


Having a global to configure a generic utility function seems wrong to me –  that leads to the same program inexplicably working on one system and not on another, and eventually finding that they've set their default tab size (or whatever global config) to something different. Global options like that are pretty much only ok for user interface stuff, so maybe the REPL code specifically can have a global variable instead of having a global for the indentation/unindent functions.


rather, both forks.


nit: could be const.


Fixes #8


@Bystroushaak It doesn't necessarily even have to be P2P, just federated like Git already is. Seems like one could even simply build that onto Git itself, by using isolated branches...


Actually all I did in this commit was re-name the flag and add some new ones.
It should have exactly the same behaviour as the old flag (hence why it includes only a 1:1 replacement)
A separate commit comes along and changes PVR core in response to the re-named flag, which is where I fix the behavour change.


Ist nicht mit der alten Alfred-Version (wie die von ), bitte default SOCKET Wert auf leer, als "" setzen.


The bug from #376 was here.


nit: All these functions could be contributed to lib/include/mmio.h (as a separate patch).


Looks good (y)


Pořád nefunguje


Have you noticed that liquidprompt has a `develop` branch. It may already have the fixes you need...


The last version of gulp will exit when there is an error. Will look into how to let it run tests on file watch.


This is not working yet :-/


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (743 of 743 tests passed)

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


[0] =>   							1. Immigrant Song
2. Friends
3. Celebration Day
4. Since I've Been Loving You
5. Out On The Tiles
6. Gallows Pole
7. Tangerine
8. That's The Way
9. Bron Y Aur Stomp
10. Hats Off To (Roy) Harper


Wonder why the additional file needs to be there for it to work.  https://meta.discourse.org/t/rails-girls-soc-banter/26875/61


Come on, there is no real development taking place anymore. Please just admit this, document everything you have so far and close the kickstarter.


The URL was be changed, if use own cluster of Atlassian!


@laughedelic science I must update Compota to Statika 2 (2.0.0-M2) I have a couple a question for you:

* Does this sbt-statika plugin is still supported?
* Is there any of its usage with new Statika?
* Where `AnyArtifactMetadata` is defined?


Nah, I think JSCS 2.0 is fine. Was just uncertain before when 2.0 would ship.


I know, but how should I proceed if I have this bug when I run the notifications branch. Its not sure that it is in the master or in some other branch. Shouldn't I try to find and fix the bug in this branch then?


nice


The usage of Task.Run is preferred. http://blog.stephencleary.com/2013/08/startnew-is-dangerous.html is a great resource for async / await programming in C#. Most of the articles from that blog support a book the that was written.


We might consider using a Factory pattern here


let's have some contribution


i liked kai :man:


True, but I love splitting refactoring only (this one) and actual code changes (the next commit), otherwise reading the git commit history gets more complicated


This commit should be reverted and removed but it was pushed when I wanted to show you the possible fix for #17. It was a part of my faultfinding of dead connections.


Appreciate that!!! thanks @bomberstudios


The logic may be slightly off here. Admins will only be able to manage collaborators for wikis the admin owns themselves.
Is this closer to what you're aiming for?
```ruby
self.admin? || (self.premium? && wiki.is_owned_by?(self))
```


@yann-papouin @yvaucher Reason : a mistake :/


`store.has('ggg')` would return true, yea
`store.has('abc')` would return false


I bet you already know about it (for you fixed the code), but just in case, here's a good explanation about AngularJS' `$apply`: http://jimhoskins.com/2012/12/17/angularjs-and-apply.html

This is one aspect of AngularJS I especially dislike, it's just too easy forgetting about it.


`create_list` can be simplified to `create` as the list size is only `1`.


The reasoning behind not storing any state in your quick fix is pretty much the same as for not storing PSI elements in your quick fix. But, of course, memory footprint is not that significant in your case and we can leave these fields be.

Anyway, you're computing something that possibly will never be used. Possibly, it would be better to extract name, arity, and parameters computation, and pass it as another constructor argument.

What do you think?


https://github.com/samccone/slack-msg-proxy running on the device


There was an issue with packaged folder structure like below:

* ./dir/table
    * folder1
    * table1.txt
* ./dir/table2
    * folder2

Now readdirSync("./dir/table") was returning "too much":
```js
[ 'table1.txt', 'folder1', '2' ]
```
while it should return just two items:
```js
[ 'table1.txt', 'folder1']
```
The "2" came from "table2" (after removing "table" substring) which was entirely wrong.

***
The commit also contains a test case.


All of the above is for /upload.php/ to look and act as /upload/


Consider changing lines 96, 103, 110 to:

```
wiki = create(:wiki, private: true, user: @user)
```

This will then allow simplification of the expectations from:

```ruby
collab = @user.can_manage_collaborators?(@user.wikis.first)
expect(collab).to eq true
```

to

```ruby
expect(@user.can_manage_collaborators?(wiki)).to eq true
```


Adding rounding for highlights


Switched the URL because of URL rewriting


These variables never seem to change. What are they for?


Adding support for repo selection


Trying out "required" but it doesn't seem to be working at the moment


Go away you multi-fab! You're not wanted here.


It means you may (should?) use [`$interval`](https://docs.angularjs.org/api/ng/service/$interval).


Ini for select dropdown for repos


Hm. yea it seems it `has` works as `hasOwn` for nested keys, which is strange.


Finds either "Sorry, there was an error uploading your file." or "Sorry, your file was not uploaded."


Added description stuff. removed Star Wars reference.


Please, avoid nesting when there's no need for it.


Why should this feature be hidden?


Create blank <a> but not add to document


Make the new <a> open in a new tab


Add classes to the new <a> so it looks like a Material button


I don't like this ```substring(0, line.length() - 1``` logic. Not only it looks bad, it's also dangerous - why not change your regular expression instead? (hint: use non-capturing groups: ```(?:X)```)


Liked the concept of method chaining in the reference


What if `expire_time_us` is near the max `uint64_t` value and `atf_sched_clock()` wraps around? You could be waiting a long time.


nit: This driver could implement the generic "delay timer" API instead (https://github.com/ARM-software/arm-trusted-firmware/blob/master/include/drivers/delay_timer.h).


I agree, there are exceptions. Perhaps "As such, there is generally a lot..." is better.


I believe this build has some slowdown issues.
We were testing with a Sony android yesterday and it took 2 minutes to add a transfer to the transfer list.
It already had 3 other transfers.


Make the <a> button say Report


Well, I know the article, but don't find it a "good" explanation, because in the end it's "You never need it except when you do", and then gives an unique example with `setTimeout()`.
So the same with `setInterval()` is obvious, but what of the other cases?

I might indeed use `$interval()` if it allows me to get rid of all the `$apply()`.


Notification comment.


曾先后在我们的


i remember there was some compilation flag that in the past would make the library really slow, perhaps it's turned on, or something similar.

I'll start profiliing today to see how we're doing.


/cc maintainer @Fuuzetsu.


Why have you marked ```rootPath``` nullable? From what I see, it's always not null.

In case you use nullable parameters in future, prefer passing them after all not-null parameters - it improves call sites readability, in a way.


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1769 of 1771 tests passed)

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


Thaaaat's absolutely strange... tests always passes no matter
`store.has('a.b.c.d.x').should.be.true;` or `store.has('a.b.c.d.x').should.be.false`; lol


As Sebastian suggested in the meeting, we should use com.intellij.openapi.vfs.VirtualFile.contentsToByteArray().


Awesome! I'll have to try that out


No. Please stop doing this, you can just declare your XDG session as "GNOME", and leave the rest to the desktop files "NotShowIn" property.


Please use `Em.empty`


Hej, Peter!

It was originally to be only a piece in my eng. thesis, but i agreed with my promoter to make pyroute2 enhancements a whole thesis. Next week i'm on holidays, i'll get to it when i'm back. You can expect tc-ematch pull request (and possibly some more ipsets) mid-August.


Nekiestem mégis az ffmpeg-2.7.2 -nek, szépen haladok.

Az opencv -t letiltottam az ffmpeg build-hoz, mert egyrészt keresztfüggés, másrészt szinte minden disztró libopencv nélkül gyártja le a forrásból. Táplálkozzék az opencv az ffmpeg -ből és ne fordítva!

Első körben most 21 ubé van kész.
A KDE cuccokat nem frissítem, kivétel a már kész k3b.

A maradék gonddal úgyis jelentkezem, addig ne foglalkozz velük.


Please place your private and static methods below public/protected methods of your class


UU

TADY TADY


@Foggalong Just wondering: is there no linear gradient for the baseplate of this icon on purpose?

Actually, there is a linear gradient in the source code of this icon above, but for some reason, that passage does not appear in the source code of the icon that I just fetched and merged via git. Also, Inkscape is not displaying a linear gradient, but rather a solid "fill" colour `#d5d5d5`. Is this s.th. on my end?

The (local) text of this icon on my end looks like this with gedit, apparently a couple of lines are missing:

![text](https://cloud.githubusercontent.com/assets/7164227/8982532/bb1a8da8-36c2-11e5-8ceb-e7a995bc633d.png)


Somewhat confusingly, `$` should be used in String interpolation instead of `%`. Also, `s"` should be used in any interpolated string. This matters in cases like this where there are actual two strings separated by a `+`.


Nossa jovem, aí estagia no paraná. rsrsrsrs


这样不也依旧是相对路径么? 绝对路径不是 `/src/views/reader/index` 这样吗?


## QUESTION/ARTICLE/TAG GET Ajax 서버 요청 연결 완료
- 스타일 시트 임시로 부트 스트랩 적용
- QUESTION 리스트 / ARTICLE 리스트 & 본문 내용 테스트 완료 **[이슈 있음]**


Agree. So +1 for this approach. Or at least, not a -1 :)


> I guess we're now all collectively realizing what a mistake it was to centralize a decentralized version control system (Git). This is what happens when you trust private corporations with your critical infrastructure.

@joepie91 That's exactly it.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 354](http://192.168.30.225:8111/viewLog.html?buildId=1775&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Code review is really useful :)


using Retard triggers GitHub, while this is totally cool: https://github.com/freebsdgirl/ggautoblocker/commit/d59ec0e06fc739e6b338f3199449c2bbeb14595a


The CSV now dows not comply with https://www.ietf.org/rfc/rfc4180.txt - but it works for excel for more languarges


Si c'est une url il vaut mieux utiliser URLField(null=True, blank=True) (et mettre un "blank=True" dans tous les cas).


¿Qué vendría a ser esto?


10 anos de experiência em estágio já... rs


EventUtil.announceToAllNearPlayers?


Name things properly - this method creates compiler messages, and it's not just 'processing text'


ici il vaut mieux utiliser "reverse", ça changera partout automatiquement https://docs.djangoproject.com/en/1.8/topics/http/urls/#reverse-resolution-of-urls


Of the library?


Perhaps a typo? http://httpd.apache.org/docs/2.2/mod/mod_proxy.html#proxypreservehost


et par convention il vaut mieux appeler la méthode "get_absolute_url", c'est appelé à pas mal d'endroits https://docs.djangoproject.com/en/1.8/ref/models/instances/#get-absolute-url


Autant supprimer ce champ alors non ?


Extract variables for all parameters and issue a single CompilerMessage constructor call - the code will be cleaner, I think.


Only briefly tested.
User handling seems to work after the patch.


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Good stuff.

I think the toplevel check is not necessary (it's inside the QgsDataItem class). Just return 0 in a else branch next to all the else if's.

sipClass_* is deprecated since sip 4.8, sipType_* should be used instead.


I felt less comma style is natural.
Now I find out that [Q and A](http://english.stackexchange.com/questions/412/should-i-put-a-comma-before-the-last-item-in-a-list).

We have no guideline for the rule of comma (and Q and A above says "Comma use is something of a grey area").
If you think we should make such guidelines for comma, I will write it down to rails guide.
BTW should I revert this commit ??


Yup.


> I guess we're now all collectively realizing what a mistake it was to centralize a decentralized version control system (Git). This is what happens when you trust private corporations with your critical infrastructure.

> @joepie91 That's exactly it.


a


Can you by any chance explain what this is supposed to be checking? It fails on my system and I'm trying to figure out why. The `-r` flag would normally take a file argument of a reference file to date, but so such files exist for me. It looks more like a time signature being passed in `$time`, but it's not understood by `date -d` either, so I'm a little puzzled...


please use an explicit return type here to avoid an isXX method that returns a Boolean wrapper


Yes, I think it could be removed now, as we has different function to mark deeply and shallowly. SER_EXT series will never be called with mark_deep.


Spamming with comments for you to be notify


sử dung isNotBlank() của CollectionsUntill để kiểm tra emtpty của list, không return về null, set password rỗng cho user trước khi trả về


max -> getResultScale


Please, show a whole command instead. Showing path to 'escript' executable doesn't provide any information to solving a problem.


The PsiClassMirror does read operations on the psiClass as well, those also need to be done in read actions


lgtm


http://3bollcdn.com/nethunter/chroot/kalifs-minimal.tar.xz

http://3bollcdn.com/nethunter/chroot/kalifs-minimal.sha512sum


why was this removed?


remove throws Exeption


không sử dung try catch


Add the link to the <a> button.
Base for new issues: `https://github.com/materialos/Icons/issues/new`
Title Parameter: `?title=%5BBug%3A%20Uploader%5D%20Upload%20Failed` Viewed as `[Bug: Uploader] Upload Failed`
Body Parameter: `&body="...`
* `encodeURI()` Makes spaces %20 and most (I did some myself below) special characters %Something in case your browser doesn't catch it
* "```\n" - Three backticks to make the body wrapped in block code
* `txt` variable that contains the console output without `\n`s
* `.replace(/</gi, "<")` - Replace HTML entity `<`s with the actual `<`
  * gi means global case insensitive aka: replace it everywhere regardless if `TEXT` or `text`
* `.replace(/>/gi, ">")` Same as above but `>`s
* `.replace(/<br>/gi, "")` Replace line breaks(`<br>`) with nothing, global case insensitive
* `.replace(/;/g, "%3B")` Replace semicolons with parsed value, global


Now my answer is 26241. Hopefully it is the correct one.:D


set rỗng password cho mỗi user trực tiếp ở trong list


Add the <a> button to the end of `.card-action`'s first child.


Do not use capturing groups when you don't use them. They waste a lot of memory.


hey @dfahlander, it would be super useful to describe what would the limitations be on a separate page and link to it here?


@kevva Why did you remove this line?


Catch git errors


More descriptions


yes, the video streaming seems to be sketchy, but not sure of the issue at the moment. I'm using a standard html5 <video> tag on the page, so I'm letting the browser handle the playback of the stream. I've notice some significant latency from the server, but not sure what the cause for that might be.

BTW, I'll be posting another commit soon to view the location data for a video


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 354](http://192.168.30.225:8111/viewLog.html?buildId=1775&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:41


Should be `##### AngularJS` :smile:


Same as previous report button (above) but with a different title


:+1:

Thanks for fixing this long-standing performance issue.

The link in the commit to Issue 320 is broken due to the migration away from google code. It can be viewed at [web.archive.org](https://web.archive.org/web/20130725013416/http://code.google.com/p/msysgit/issues/detail?id=320).


bad idea ... you should do this via CacheController.ControllerBehavior


Please, avoid nesting. Doesn't this code look cleaner?
```java
    Matcher matcher = LOG_MESSAGE_PATTERN.matcher(StringUtil.trimTrailing(message));
    if (!matcher.matches()) return null;

    String type = matcher.group(1);
    String details = matcher.group(5);
    return new RebarMessage(findCategory(type), details);
```
You check your preconditions first, and then do your stuff. I hope, it makes sense.


I'm trying to compile the new test for block2...
The compiler complains about missing `triqs/gfs/impl/block_gf_iterator.hpp`


Method ordering.


sử dụng findByEmail để thay thế cho findAll, không gắn getus = null


Nullability annotations.


Sorry. pushed.


Please use CancelIndicator.NullImpl instead to stay consistent with other locations where a dummy CI is used


Really complicated regex code highlighting will update later.
green: success or addition
blue: value
yellow: change
yellow background: minor error
red: deletion
red background: major error
purple: string or something enclosed in `[` `]`


Added `/upload/index.html`'s top to upload.php


Added `/upload/index.html`'s bottom to upload.php


Bye, bye `/upload/index.html`


Shouldn't this be either an Error or a ProcessCancelledException?


We don't use this kind of spacing in arrays (I'm aware that master has it as well).


It information will be duplicated because we show whole command as information when command is running.


Yes, because we use this data in `article-wrapper.hbs` like that:
```
{{#each headers as |section|}}
	<li {{action 'trackClick' 'toc' 'header' on='mouseDown'}}>
		<a href="#{{unbound section.id}}" title="{{unbound section.name}}">
			{{unbound section.name}}
			{{svg 'chevron' viewBox='0 0 12 7' class='icon chevron'}}
		</a>
	</li>
{{/each}}
```

We could extract this data later in `ArticleWrapper` but let's not do it now.


for this plugin, we don't need the instance id


No, this is not an idea test. And our default platform specific exception is OCE.


Use ```java.lang.AssertionError```.


The schema for thm-trafviz


GROS COMMIT!


Thanks for the review ! I'll change that and push


@ercanozkaya Docblock typo


I'd argue that RebarProcessAdapter is not ErlangCompilerProcessAdapter. Consider extracting and reusing logic contained in ```org.intellij.erlang.jps.builder.ErlangCompilerProcessAdapter#processText``` instead.


:sob:


I don't see why you introduced an abstract class, whose only implementation is in the same file.


không sử dụng new Answer cho những trường hợp này
ví dụ
```
User userTest;
List<User> usersTest;

@Before
public void setup(){
    //khởi tạo dữ liệu cho userTest
   //khởi tạo dữ liệu usersTest

when(userRepository.saveany(User.class)).thenReturn(userTest))
}
```


Remove exeption


这个是什么用法? 和 `bookmodel` 是什么关系呢?


Nesse exercício podemos alterar o title da página para Ceep.


Also there are method ordering issues and unset nullability annotations in this file.


Thanks, I was about to ask for a release :joy_cat:


What is "required" supposed to do?


Aww


It should make the input files required before going to the form action (upload.php). But it doesn't seem to be working.


A `Streamer` contains `stdin`, `stdout`, and `stderr`.  If those are set, why would we want to leave some unbound based on boolean args instead of building the `Streamer` that we want?


Oh, I see. Why showing path to 'escript' then? Issue something like "rebar process finished with exit code **"


I don't know. Maybe test like "is_final" should be in dedicated functions rather than exposing the state object. I didn't have an opinion on this and this kind of test is rare, I only used it once.

If you like this helper I can write the missing translation. Beware English is not my native language!


@Ekrow that's horrible.


Lol it just seems rude if they don't know what it is.


I suppose... :(


handler_mk1.initialize(widget_mk1.component, data=None) can also be used.

SettingsHandler.initialize can initialize components as well as widgets.


TODO: add unit tests


I think I changed this in a later commit. Anyway, I am writing this to you over Greenland!!


let's do this after setting up logging and do at least a debug level message saying that that pgpy was not found and thus disabling encrypted env vars.


Doesn't it feel like these tests readability declined after your change? Do something about it.


As of df45a4d ("default to use per-repository buffers", 2014-05-05), the MODE argument to magit-mode-get-buffer and magit-mode-get-buffer-create has not been used.


Do you think the average user will understand the distinction between `attach` and `exec` or does it deserve more explanation?


Chưa có unit test cho service


okです


Probably an oversight.
Thank you, I will look into it ...


Having more than one argument here should be error.  You can't reliably complete the `AttachOptions` object with the wrong number of args.


Thanks for writing tests!


you can use `Random` if you want to, but this is quite ok


Oh, that's great! Thanks a lot!


I couldn't leave `backtrace()` as it was, because it added the exception twice and left out the message, so I moved the "Cleanup error message" commit into `backtrace()`.
It looks a bit cleaner now.


yup :+1:


You can't do this on existing repos. It only applies to new ones.


@jordansissel why is stdout/stderr preferred over log/err? Thanks.


No problems, drop me a line if you need spanish translations for your projects


Thanks for investigating! :heart:


Just noticed that this made the build to fail. :/
https://travis-ci.org/inveniosoftware/beard/builds/73154775


Should this be `X-Spam-Score` to match the default and the documentation in the comment above?


This library uses camel casing specifically to match as closely as possible the canonical JS client.  This is by design and I don't want to start changing things midstream.


sử dụng phương thức post


as above, please use encryptEnvVarMessage


and encryptEnvVar


@kyon0304 以 src 为路径啦。 因为 webpack.config 里头设置了 src 喂resolve 的路径。所以这样可以读到。。。相对好像是 ./ 这种开头的把。

这么做的目的是 。如果你在view组件里头以 `../models/canvas` 这样引用，如果 这个组件被移动了。就要修改相当多地方了。 要改成 `../../models/canvas` 才能运行


Hi, I would like use IR with ATtiny84.
I use this  Libs in my program.
#include <IRremote.h>
#include <IRremoteInt.h>
but when i compile for ATtiny 84@ 8MHz ( internal Oscillator;BOD disable) or ATtiny 84(intrenal 8Mhz clok)
i have

/Applications/Arduino/Arduino.app/Contents/Resources/Java/hardware/tools/avr/bin/../lib/gcc/avr/4.3.2/../../../../avr/bin/ld: ventilo2.cpp.elf section .text will not fit in region text
/Applications/Arduino/Arduino.app/Contents/Resources/Java/hardware/tools/avr/bin/../lib/gcc/avr/4.3.2/../../../../avr/bin/ld: region text overflowed by 1152 bytes

have you a tuto for exemple to ATtiny84 ?
thanks


1: M111 unrecognized component name: WebUser


@sindresorhus Oh, is the container-based infrastructure enabled by default now?


Maybe to add `tooltip` here as we have it on right side bar.


In the long run, does it make sense to create a single Task continuation where we convert the normal before() behaviour to an async task? If we do that, we could use a single chain to run all before actions, all act actions and all assertions as well. Almost something like this -- again, not 100% sure how it would be implemented.
```
void BuildContinuationChain() {
  ///psuedo code here, as I'm not 100% familar with the inner workers of the runner
  if (Context.before != null) { _continuationChain.ContinueWith(() => Task.Run(() => Context.before(..))); }
  if (Context.beforeAsync != null) { _continuationChain.ContinueWith(() => Context.beforeAsync(..)); }
...
  if (Context.act != null) { _continuationChain.ContinueWith(() => Task.Run(() => Context.act(..))); }
...
  foreach(var test in it) ///get assertions here
  {
     _continuationChain.ContinueWith(() => Task.Run(() => test(...)));
  }
...
  _continuationChain.Wait(); //or something to pause until the tests are complete for that specific chain
}
```

Again, I'm not 100% sure on this design, but it feels like its heading in the right direction. It also allows a non async method to be executed and may eliminate the deadlocking issue. I'm also not 100% sure how the _continuationChain would get started, or if it is just a Task.Run() itself. What are your thoughts?


không trả dữ liệu null
ví dụ
```
public User save(User user)throws Exception {
		if(user == null){
		    throw new IllegalArgumentException("user could not empty");
 		}
-		return null;
		User userCheck = getUserbyEmail(user.getEmail());
		if(userCheck != null)
		{
		    throw new RuntimeException("user duplicate");
		}
		return userRepository.save(user);
 	}
```


Seems like a more direct test of what we're expecting might be to test that the relevant request has received the email.


Girl:Girl?


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 355](http://192.168.30.225:8111/viewLog.html?buildId=1782&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Ahh, if we do this then the logging above might be extraneous.  I still thing a debug level message on import is not unreasonable.


Why you call it stupid?


Just moved this into the cell swift file - because it is in every cell we can do it in the initialisation code - just semantics :)


Yes, but only for new repos.


Ah yep; must have missed that one when fixing up


`modelAndCreateFuncMap` 是 `APPModel` 中一个数组类型的属性? 在哪里定义的?


It's in a subordinate clause and is a gerund (or something like that). Sounds correct to me (native UK English speaker).


The name of this function is not very clear as to what it does.  Can we give it a different name?  maybe just `messageForEncryptedEnvVar`.  If it's only used internally in utils.py, please prefix with `_`.


And sounds more natural than the previous version.


Chưa có unit test


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1261 of 1261 tests passed)

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


@kyon0304 正E文件的第五行


@louh I revert this change to update the widget position to the active key. That was what was breaking yesterday


设置了 get 函数, `func` 不是一定等于 `{canvas: 'createCanvasModel'}` 吗?


Cool @Orbiter


@kyon0304 es6 的 class 不能写属性。


It's artefact from dirty changes, I will remove this annotation.


OK


@hxgdzyuyi  那个不是 `getter` 函数么


@kyon0304 这个用来管理全局变量的


@alex-deaconu sorry, I got confused with the apps jshint setup. cheers


Thanks for hint. I fix it.


no ones likes kai.


Consider running a search for "concrete" and drop it where you can. In this case, it does not bring much value to the table so you can safely remove it =)


IMHO `TaskDefinition` object is part of the deployments infrastructure, so it should contain a `validation_status` property for the represented task.
If we go this way, it should also contain the functional accessors (like `validate`) to get more information from the object.
Basically I want this object to describe the status of this task in a context of the current deployment. The `Validator` (which is a great idea by the way) should be talking to  this description only.


@hxgdzyuyi 嗯嗯


Ok


@kyon0304 是啊 get 函数就是设置属性吧。？？


Would it be a lot of work to add support for decrypting environment variables to utils.py?  I could see a future worker using this client and wanting to decrypt environment variables.  Also, it would let us have end to end testing to ensure that the environment variables can be encrypted and later decrypted.


Please always adjust the cfg['Lumi'] entry according to the int luminosity of the json file


@louh I turn widgets position update every time codemirror re-render. This is extra work... we should talk on how to detect changes on the text and only repair widgets with keys on changes as you suggest


In a few commits we start using it again, but more as a hack than anything else.

It would probably be a good idea in the long run to find matching buffers using the major-mode and default-directory and completely ignoring the buffer-name. Currently we create "sticky" buffers by renaming them, we probably should instead mark them as such using a buffer-local variable (and still rename the buffer, but that would no longer influence whether this function returns that buffer).


1: M190 line is too long (51 > 50)


@Ekrow That's hilarious.


0/20 structure.


An our has slipped in here. Try going with "Typically, the NativeScript definition files begin with the API first..."


has => have
(since "posts" is plural)


Wow, why is that reflection hack needed?


Enclose \_\_extends in code formatting

```
`__extends`
```


This is a good idea. And it shouldn't be too hard to add.


calculates its core => calculate it's score.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1261 of 1261 tests passed)

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


double the (how the the getter and setter)


Whitespace.


Weird. I thought tagging this commit in mine would auto-notify.


@fmallet you didn't update the license in the plugins' pom.xml


確認しました


This will always be `true` statement. You have to check length of returned array :]


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 355](http://192.168.30.225:8111/viewLog.html?buildId=1782&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:32


could not replicate slowness on OnePlus...


Those handlers will never be called, as only [`/api` is registered](https://github.com/pbochis/app/blob/master/app.yaml#L28-L30).


Please use braces consistently; if the `if` condition has braces then `else` should as well. The code style guidelines for AOSP [require][standard brace style requirement] braces on all conditional statements unless the whole thing is in one line.

[standard brace style requirement]: https://source.android.com/source/code-style.html#use-standard-brace-style


`parseInt()` should always be given the second arg (radix), which defines which number base to use.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/parseInt


No, commits don't receive auto-notifications AFAIK. Just issues, PRs and people.


Ticket created for the next sprint: http://jira.sonarsource.com/browse/SONARTFVC-23


Good to know, thanks :-)


internals are for internal variables.
Do not want to use internals for required modules included at the top of file.
hapijs style is to require packages or modules at top of the file and the variable made with these
required files are capitalized. You want to below:
```
var Package = require('../package.json');
```

After you do the above, you will not need:
`internals.version`  or `internals.description`
The will become Package.version and Package.description

See this for example: https://github.com/hapijs/university/blob/assignment2/lib/index.js

Assignment just required one route.


parens


"Since it is granted this code is available for Android only" is a bit weird documentation-wise. Try "Because this code is applicable to Android only, you can safely..."


格式问题


`.min` file that doesn't seem to be minified?


It's `false` by default now :).


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


OH SHIET.


Could you explain this change.

Before this change we used `Serializer.root_name` as `json_key`. Root name defined as follows:

```ruby
    def self.root_name
      name.demodulize.underscore.sub(/_serializer$/, '') if name
    end
```

So for `TestMovie` and `TestMovieSerializer`:

method | value
-----------|-------
root_name | test_movie
json_key | testmovie

What is the aim of both methods?


> Thanks for fixing this long-standing performance issue.

Is it a performance issue? I thought it was an issue with being denied to execute certain commands. Do you have timing statistics? I would be very interested to see them if you do.


Could we move this out into its own file?


test branch gh-pages


Raising the "since" vs "because" concern. Since is more popular as an indicator of time passed -> "Since 2002, we have been developing quality UI components." Where you're using since as a replacement for because or due to, replace it with because.


Two days of investigating, and one line to fix it :smiley:


_page will start from "0"


![screenshot_2015-07-30-18-29-12](https://cloud.githubusercontent.com/assets/8209158/8983504/63b1d70c-36fd-11e5-86e2-c6d7536d55c3.png)
![Uploading Screenshot_2015-07-30-18-29-23.png…]()
![Uploading Screenshot_2015-07-30-18-29-55.png…]()
![Uploading Screenshot_2015-07-30-18-29-30.png…]()
![Uploading Screenshot_2015-07-30-18-30-21.png…]()
![Uploading Screenshot_2015-07-30-19-36-49.png…]()


I think these newlines at the start of classes and methods should be removed, as they just expand the overall size of the code.


RedisClient.connect accepting codec and RedisURI is private. I stumbled upon this issue the 2nd time today. I guess, I should expose that method publicly in lettuce 3.3 which comes end of august. Btw. What's your schedule for 5.0?


In this way memcached is always defined and so it's always enabled, but this should not be the case. Or am I missing something?


@hxgdzyuyi 窝可能没明白这个函数的意思,  给 class 添加属性吗? 还是给属性赋值?


ReadVarsSmallDiffTest (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1759 of 1777 tests passed)

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


Here it is sir


why just don't use org.jboss.windup.maven.indexer ? Isn't it too long to have org.apache there?


This could be problematic for all the `execute` commands as they don't support callbacks anymore. Using `.then` should fix the problem I think.


Get-CloudWatchLogs.ps1 commit.


Preserve the numbered list here, if this is indeed the required sequence of steps. What I meant with my previous comment was that you can mark your items with just `1.` (no need to take care of the ordering yourself - Markdown takes care of the ordering for you).


sorry :disappointed:


Is there a lot of shared knowledge about this library in the releng team?  I'm not familiar with it and want to make sure that if it breaks that we have a couple people who understand it.


These need to be indented to be considered parts of the step they belong to and in order not to mess with your numbered list output. If you leave them as they are now, the index of the next step will be restarted.


AMIT amit amittttttttttt 222


It's hard to remember why, but it may be because of the addressable pinned to 2.3.5 version. However, in my new web archiving WF, I removed the pinned for the addressable and nokojiri and both work fine.


typo - should be "welcome". (I can't edit this file yet - not in master branch?)


Don't hard code data.


It also strangely behaves with namespaces models:

```ruby
class Admin::User
  include ActiveModel::Model
end

Admin::User.model_name.to_s.downcase #=> admin::user
```


Again, in order to have correct indices, you need to indent this code block.


Seccond


What do you mean by "the Native world"


Güzel olmuş, aferim


Should this be `if (! StringUtils.isBlank(this.defaultAuthenticationMethod))...` - not isBlank branch?


What you need to do is to create... -> You need to create...


Where is this dependency used? I don't spot it in this repo.


This is a method on the `Proxy` class which synchronizes properties...


reopen.


OK, so:

* the title of the page is slightly misleading :-p
* we need to add Docker to the services list. (Any default config?)
* Do we support database use on OSX without using the service mechanism to do so? Do we need to tell people about it?

I'll work on that wording.


@evdokim cool ") here are my answers:

In the end we decided not to use the plugin. The intention is to simplify things. Actually, I think it's still no the final decision, but at the moment the picture is as follows:
- bundle projects don't need anything from here (so they use only nice-sbt-settings plugin). see for example the [bowtie bundle](https://github.com/ohnosequences-bundles/bowtie2/blob/master/build.sbt) and others in the [ohnosequences-bundles](https://github.com/ohnosequences-bundles) org
- distribution projects have this metadata generation code in their config _explicitly_ (not nice, wasn't enough motivatin to mantain a plugin just for that). so far we have one such project: [era7bio/bundles](https://github.com/era7bio/bundles/blob/master/build.sbt)

And `AnyArtifactMetadata` is defined [here](https://github.com/ohnosequences/statika/blob/master/src/main/scala/ohnosequences/statika/Bundles.scala#L86) — nothing special, actually, statika uses **only** `artifactUrl` from there (to download the fat-jar on instance).


Told you it'd be easy. ;)


I used the `appsto.re` url, because that's what I believe the App Store uses when you try and share a link. Although, Apple have changed this URL over the history of the App Store. They used to use the full link, so maybe it's a good idea to include it as well in case they change it again in the future. I'll update the code.


If one creates SEARCH_BACK_MS variable in heroku, would it override the default?


在这里
```js
var canvasModel = new CanvasModel(bookModel.getCanvasModelInfo())
app.setModel('canvas', canvasModel)
```
是不是比在 `app.js` 中 `createCanvasModel()` 更一致些

不过窝没有明白 `createModel(name, ...ohter)`  做了什么..


I have almost no idea of Python and this is exact the thing i searched for, so loooong today =)


I know, honey bunches of oats


Sad to say but you're probably right @douzepouze ...


Ok.


@dgageot I hope this URL is an API because I'm going to use it from AppVeyor :)


let's do something like

```javascript
expected = {
}
actual = subject.....
self.assertDictEqual(expected, actual)
```
to improve readability


What do these numbers mean?


I used this widget only inside another widget, so for me Widget.createWidget worked, but you are right.


extern?


It is used by mock https://github.com/testing-cabal/mock/blob/master/mock/mock.py#L83

it is necessary on python 2


为什么不需要 `app.createModel('book')`..


es5 可以这样写

```javascript
var DocumentRow = Backbone.View.extend({

  tagName: "li",

  className: "document-row",

  events: {
    "click .icon":          "open",
    "click .button.edit":   "openEditDialog",
    "click .button.delete": "destroy"
  },
})
```

但是 es6 并不能啊

```javascript
class xxx {
  xxx : {} //这是错误的。。。
}
```


Show!


Давай вместо переопределения метода mapRow класса RowMapper прямо в вызове функции jdbc.query(), отдельно сделаем наследника, например FlowerRowMapper и передадим в функцию его объект. Так будет четче видно, какие аргументы принимает функция jdbc.query()


Overall looks really good, thanks for writing this!  Once we address the naming, I think I'm good to merge.


This test is written at the wrong level at does not belong here.

If you want to test the compiler, it should be packaged as a call to `unexpectedType` (in order to call the type checker).

If you want to test the evaluator, it should go somewhere else, e.g., `KeywordParameterTests`


This seems to have broken manta on tumor-only data, see #952.


Thanks!


I think this check shouldn't be part of this function.
If a wrong file passed to `afunix-source` then I think we should check it _before_ even we call `afsocket_sd_init()` method.


I think this check shouldn't be part of this function.
If a wrong file passed to afunix-source then I think we should check it before even we call afsocket_sd_init() method.


Hey, this needs to still be checking for the badge type boxes - otherwise it hides the payment data on admin forms.


What the XDDDD


Maybe it could be reported as a feature request (or exists even), I don't know.