oh I thought it was a basic object too, yeah so it should actually be:

``` ruby
super || builder.respond_to?(meth)
```


Hi,

the implementation for this is at https://github.com/puppetlabs/puppetlabs-azure/pull/87 and still needs to be finished up.

Apologies for the late answer, this is not a usual place to drop issues. Please file tickets in the MODULES section of our JIRA at https://tickets.puppetlabs.com/browse/MODULES/ so that we do not miss important issues, and the whole team can pick it up.


Hi,

while there is no specific test to refute your issues, this looks like you haven't deployed the most recent code to your environment, or have not reloaded your puppet server.

I've added https://github.com/puppetlabs/puppetlabs-aws/pull/318 to add the missing test. This is passing on my local machine.

Apologies for the late answer, this is not a usual place to drop issues. Please file tickets in the MODULES section of our JIRA at https://tickets.puppetlabs.com/browse/MODULES/ so that we do not miss important issues, and the whole team can pick it up.


user_ed - лучше userEdit или userEd или uEdit, просто чтобы не было подчеркивания.


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2230 of 2230 tests passed, 12 test warnings)

 * 11 tests had: ERR diffs.

![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)


Hi,

The tests in spec/acceptance/autoscaling_spec.rb clearly demonstrate that the `tags` property is working. This looks like you haven't deployed the most recent code to your environment, or have not reloaded your puppet server.

Apologies for the late answer, this is not a usual place to drop issues. Please file tickets in the MODULES section of our JIRA at https://tickets.puppetlabs.com/browse/MODULES/ so that we do not miss important issues, and the whole team can pick it up.


timeseries-2_0 (jasondegraw) - x86_64-MacOS-10.10-clang: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


:+1:  Nice solution!
For a next step, try to achieve the same functionality with only one function in your JavaScript!


For Compatibility purposes you could probably create an overloaded methods instead of modifying the old one


Ok


Hi,
when I am trying to build specific projects I get error:

build/mvn -DskipTests package -pl examples,assembly,core,common/network-common,common/network-shuffle

[ERROR] Failed to execute goal on project spark-network-common_2.11: Could not resolve dependencies for project org.apache.spark:spark-network-common_2.11:jar:2.1.0-SNAPSHOT: Failure to find org.apache.spark:spark-tags_2.11:jar:2.1.0-SNAPSHOT in http://repository.apache.org/snapshots was cached in the local repository, resolution will not be reattempted until the update interval of apache.snapshots has elapsed or updates are forced

If I do full rebuild then it works:
build/mvn -DskipTests   clean package

Have you encountered with that?


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1114 of 1114 tests passed, 1 test warnings)

![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-34.65%25-green.svg)


@wolfylambova22, sorry, but in the description of your task still specified both.


@moeamaya It's called semver, btw :bowtie:


Hello @brandon-barker! It's me again.

I am receiving the following error when trying to install the new `0.1.2` version:

    λ bower install angular-float-thead --save-exact
    bower angular-float-thead#* not-cached https://github.com/brandon-barker/angular-floatThead.git#*
    bower angular-float-thead#*    resolve https://github.com/brandon-barker/angular-floatThead.git#*
    bower angular-float-thead#*   checkout v0.1.2
    bower angular-float-thead#*     invalid-meta The "name" is recommended to be lowercase, can contain digits, dots, dashes
    bower angular-float-thead#*         resolved https://github.com/brandon-barker/angular-floatThead.git#0.1.2
    bower angular-floatThead#0.1.2     ENOTFOUND Package angular-floatThead not found

Any idea on what may be causing this? Meanwhile, I will keep using my branch.

Thank you!


В аргументах функций переменные лучше называть с маленькой буквы.

HttpGet.getData($http, $q, './src/json/group.json')
$q - не нужно передавать в метод - это создает замыкание и забирает память, которая не будет очищена, пока существует сервис. А т.к. сервисы это синглтоны, они существуют, пока работает приложение - т.е. память не будет очищена никогда.

Если нужно использовать сервис внутри другого сервиса, тогда лучше его просто заинжектить.


A w tę stronę jak najbardziej, nie pomyślałem w ogóle.
Pytanie czy należy to sanitazować przy wyświetlanium, zapisie do bazy czy w obu miejscach.


I mentioned the performance problem here as well, needs to be fixed before we merge.


I'm a dummy, what does the NoolsTest bit at the start do? Does it mean you can either use it as a module or just import it and have it set this value globally?


if (!data) {...}

И не забывай ставить ";" в конце строк.


cc @mrose17 pls fix (and other comments about xhr)


Yes!


@jdurgin I think it may depend on how accurate that we want the value of _buffer_size to be. Is it ok that I double check _buffer_size in Sync(), like codes below. The benifit is making the critical region smaller. Which one do you think is better?

if (_buffer_size > _env->_write_buffer_size) {
    std::lock_guard<std::mutex> lock(_mutex);
    if (_buffer_size > _env->write_buffer_size) {
        // do sync
    }
}


sudo_users = {83135960},


`强烈建议将英文原文也附上，放入另一个目录。`
>好处：
>如果出现翻译错误，或者有争议的翻译点，可以参看原文进行确认


This is really weird syntax--an array with key values? How does this work?


This clones the array, preventing `shift()` from modifying, which we don't want as calls recurse.  An alternative would be to say `key = keys[0]`, and then `keys.slice(1)` in the `*` block.


@jfirebaugh - because hard-coding and assuming `std::vector` is a bad c++ style ? Please, explain what is your problem with OGC/ISO standard in this case :

<img width="710" alt="linestring" src="https://cloud.githubusercontent.com/assets/59056/16808344/10a9faf6-4914-11e6-8ce6-f5a8d4fb3dc1.png">


design looks like a cross-over between the Kestrel and the Autumn Leaf


Deju en producción


holyshiz this is a huge merge. xD.


(fixed)


Fixes possible crash


@mhartington and @ihadeed thanks for immediate response to my question. Unfortunately I will have to agree with @keithdmoore. Having a declared set of possible values gives the user a much better understanding of how to use the plugin without having to read the documentation of what the possible values are and I don't want to mention hair pulling cause of typos in string comparisons.
In my case I have included the Connection class in my own code.


I also get NPE when building


Т.к. у твоя задача сначала загрузить группы, а уже потом юзеров, имеет смысл убедиться в том, что порядок их загрузки будет сохранен. Тогда можно отрефакторить код следующим образом:

UsersService.getGroups().then(function (groups) {
    this.groups = groups;

    return UsersService.getUsers();
}).then(function (users) {
    //...
    this.users = users;
});


it reverts notebook tabs to gtk2 design.open twf to see it.no need for a screenshot.

the scrollbar part you can skip, cause its only making sense in cinnamon.
do also look at. toolbar.primary-toolbar cause .primary-toolbar is definitely not defined in 3.20.
there was something about progressbar i corrected, compare current github from me with lp.


I did also update to the MCT19Z vendor image in addition to the above, but still getting the error.


Ah, got it--the hash is implicit, so:

```ruby
pry(main)> [hello: 'test', hello2: 'testing']
=> [{:hello=>"test", :hello2=>"testing"}]
```

I find that to be very odd, so I committed https://github.com/cehdeti/spree_gotomeeting/commit/f95d5135ca68c7a611fe586434a6dd277193df04 just to make it more explicit.


still?


Yep.  Actually not used currently as a module, but seems like a reasonable to keep it to express what the public API is.


what is this `Windows` file?!


sudo_users = {83135960}


@SCdF this is where the global's injected into the test script


done.
Also updated doc of Walloverlap


GH logged @{b69527d8-a558-4bb4-8488-e0b8f745a29f,tu2}


yeah still fails:
`                        noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));`


lool - In fact, I'm currently very busy building couple projects.. and relying on LaraDock! that's why I can't have time to try new things and break things up.. as it will affect the deadlines of my projects..

However, me and the ~600 unique cloner will appreciate if you fix that problem anytime soon.

I will test and merge your PR as soon as possible.
_(I've already listed your name between the Main Contributors on the readme)_

Thanks again.


@{b69527d8-a558-4bb4-8488-e0b8f745a29f,tu2} reply


**Performance test:**

* test step 1 and 2 for in-out, repeat for in-out & in-in (use cosine for step 2):

Step1, in-out: 85 s
N pairs: 253820655
Step2, in-out: 265 s
N pairs (Introduced only): 253855975

Step1, in-out & in-in: 83 s
N pairs: 457909452
Step2, in-out & in-in: 488 s (O(N^2) complexity would bring us to 795 s)


ok


I just noticed this it's from an earlier commit:

if [ "${error}" -eq 101 ]

this needs to be either

if [ "${error}" == "101" ]   # quotes added to 101 and string comparison operator
if [ ${error} -eq 101 ]       # quotes removed and interger operator

this happens in other places as well


I'd love to tackle that PR myself, if it's not too complex.


I think we actually don't want to run the force algorithm on drag, but only at the start, leaving then all layout alteration choices completely in the hands of the user (and when importing new items, we'd probably want to hold existing nodes in place and autolayout only the newly added nodes)


sudo_users = {83135960},


@trevj While working on #2292 I think I may have found a bug that's been here for ages: Should this have been `.substring`, not `.substr`? "The difference is in the second argument. The second argument to substring is the index to stop at... but the second argument to substr is the maximum length to return." If so will fix on my branch.


In Arma 2:DayZ Mod we had an option like "Take Uniform" on dead bodies. This is not a valid option?


I will move them to vendor tree. But these blobs is difference from stock.
And omx components sources only work with them.


Hi @lengerfulluse , if I remember correctly there were indeed problems with non-ASCII characters. Ruby threw an `ArgumentError
(invalid byte sequence in UTF-8)` exception. I assumed that `tolower()` was responsible for creating those invalid byte sequences but that is not possible, as you have shown.

What about hashing the strings and using those hashes as header IDs? SHA1?
But URL encoding would be fine as well. I don't have much time at the moment, so please go ahead and create a PR.
If you want me to create the PR, I can look into it next week.

Please let me know how you want to proceed.


not a good name 💩


Thanks, fixed.


Thanks, I'm using it daily and it's been super useful!


Oh, because we're doing multiple recurses (so reusing the `keys` reference multiple times) out of the `map`, so otherwise each `e` in `element` would chip away at the toplevel `keys`. Choice!


Neat!
If you think about it, you can easily reduce the script into just one function!
For DOM events like `onclick`, you can supply parameters, even from HTML.
Check this out: https://developer.mozilla.org/en-US/docs/Web/Events/click


Hm, this seems too big and allows writing non-CPS programs...


@jdurgin Sage told me that the difference between flush and sync is that flush asynchronized write data to RADOS while Sync do it in synchronized way. Maybe I need to double check with him.


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2264 of 2264 tests passed, 12 test warnings)

 * 11 tests had: ERR diffs.

![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)


@iainlane but lets define view too, for nemo and nautilus.i don't use backdrops much..


The Travis build is failing due to this: "Double requirement given: envdir==0.6.1 (from -r requirements-dev.txt (line 3)) (already in envdir==0.7 (from -r requirements.txt (line 12)), name='envdir')"


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


2 issues here.
*Magic String* 🎅
and it should be a port *Number* not a string ⛄


Why do we need to use master here?


It's useful for me to ring a phone remotely from the cli with :
`channel originate Local/1234@default extension 666@xivo-callme`


same problem here


if you are going to define a variable `host` on line:28
use it here on line:31 instead of hardcoding `localhost`


and now loggedin


Link to Github styleguide was outdated.


@easywebhub/vinaas-core-team  sau khi suy ngẫm, anh đã tách độc lập giữa bac-si và phong-kham và kết nối giữa chúng qua slug, sử dụng 2 vòng loop để hiển thị thông tin phụ thuộc

xem thay đổi bên trên, chú ý

```
{{#each danh-sach-bac-si}}
                      {{#each ../collections.bac-si}}
                        {{#ifCond this.slug ../this.slug}}
```


sudo_users = {83135960},


It is very useful for troubleshooting calls on trunks when we are not on site.


Bad move


This is not semantically correct HTML.
`<table>` tags need certain child tags to function and display properly and should be used only when you want to present a table of data to the user.

Consider using some other block elements.


Yes, it was. It's necessary because idea module does not depend on idea-live-templates


for this i prefer to simply copy the MT_Matrix4x4 and move its origin instead of modify the global matrix.


> Having a declared set of possible values

If that is a concern, the value that the plugin return hasn't changed since 4G network became a thing.

>how to use the plugin without having to read the documentation of what the possible values are

But how would know to import the connect class without reading the docs first?
_Only half joking, still a point to consider_

We state the return values in the Network doc, and since the plugin only returns a string, it just doesn't make sense to have an additional class.  Feel free to provide your own, but the code is not necessary for the network plugin to function.


Neat solution!
For something advanced, try to implement the same functionality with only one function!


lưu ý cú pháp  ```../collections.bac-si```   và   ```{{#ifCond v1 v2  }}``` mới định nghĩa thêm trong helper


complete stacktrace pls
is this in eclipse or just gradle build from console?
did you perform clean in eclipse/gradle?


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


reply when logged in


if you are only going to invoke a single function **in** your route handler, passing the same req, res args
then you get to use your function **as** your route handler.

```
app.route('/buzzword')
.post( buzzPost )
```
👯


What do you mean "difference"?
Didn't we have to get them through extract-files.sh?


等我有时间的吧


Hello, I only pulled this repo about 3 hours ago so take my input with a grain of salt (cool to see it's so active though!), but I'm using Docker For Mac Version 1.12.0-rc3-beta18, and when I first ran `docker-compose up -d nginx mysql` it wouldn't start for me with the paths pointing to `/var/...`, I had to change them to `./data/...` as above and it then worked fine.


`
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt>gradlew build
:compileJava
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\AlertMessenger.java:27: error: illegal start of expression
                noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\AlertMessenger.java:27: error: illegal start of expression
                noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\AlertMessenger.java:43: error: illegal start of expression
                noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\AlertMessenger.java:43: error: illegal start of expression
                noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\ImageCropperTool.java:107: error: illegal start of expression
                        noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\ImageCropperTool.java:107: error: illegal start of expression
                        noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\ImageCropperTool.java:122: error: illegal start of expression
                        noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
...\programmierpraktikum-abschlussprojekt-xxxmathxxx\tddt\src\main\java\org\xxxmathxxx\tddt\gui\ImageCropperTool.java:122: error: illegal start of expression
                        noTextDialog.getDialogPane().getChildren().stream().filter(node -> node instanceof Label).forEach(node -> ((Label)node).setMinHeight(Region.USE_PREF_SIZE));
                                                                                ^
8 errors
:compileJava FAILED

FAILURE: Build failed with an exception.

cleaned in eclipse& gradle
`


It's not needed java arrays are covariant, there is no compatibility breakage any code passing a String[] will work as before


闪电在处理区，不过因为ignoremove的原因没有在界面上移动


No. I get them from sprd open source tree.


indentation


В этом нет смысла, т.к. полностью дублирует `default`.


Opened #2553 to track this.


this file can be deleted from the branch. it's generated automatically during the build configuration process.


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


Hi @toolmmy ,

Below is the error messages

`> WebSocket connection to 'ws://localhost:9876/' failed: Error during WebSocket handshake: Sent non-empty 'Sec-WebSocket-Protocol' header but no response was received`


Why was this code in there in the first place?


I think this change is somehow related to https://issues.apache.org/jira/browse/ZEPPELIN-1169. @jongyoul, could you take a look https://issues.apache.org/jira/browse/ZEPPELIN-1169?


rename to get_delta


its a good idea but lines 33 -42 will always return nil. The payload values aren't  formatted with the _id like payload[:url_id] it is  payload["url"].  like on 19


侵略的triggerable你写的有问题


The following doc(s) have been deployed to the [1621workfl](https://1621workfl-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/pantheon-workflow](https://1621workfl-static-docs.pantheonsite.io/docs/pantheon-workflow)
- [source/docs/assets/images/test-env.png](https://github.com/pantheon-systems/documentation/commit/6fb62d265bc163842d6b2d55ea284c5deee7cfae/source/docs/assets/images/test-env.png)


`class SecondPage extends Component {`


@neha-ojha this doesn't seem correct. if the tree builds (except for the class plugin we are hacking on) then that means we just need to add an include path to the build of the class plugin. if the tree doesn't build, then its probably a transient error and you can just do a 'git pull'.


Did you intend to recommit all of yesterday's changes lol


This is done with intention https://github.com/emoon/ProDBG/blob/master/src/plugins/dummy_backend/dummy_backend.c#L400 as it makes it easier to test/implement https://github.com/emoon/ProDBG/issues/242


`import React, { Component } from 'react';`


remove this


`const { appState, children } = this.props;`


Use best practice naming approach "public static type"


So then why delete the conditional check for ToneBase?


I went through tagging and I agree that it is a better way of implementing segment entities. But then when I tried to access the source id of the segment entities in the example file, it gave me a value zero . The zeroth entity was in the file was an event entity. So I am not able to access the source analog signal. So for the time being I am storing the whole dataArray returned by get _data() function. (https://pythonhosted.org/neuroshare/)


ok


Does it make sense to you? Is this behaviour what you would expect, or would you find it annoying? I think I would. The best thing to do is to imagine the bug report you'll have to address when a user says: "I clicked on something to favourite it, but instead I got redirected to its page? But the last time I favourited something it worked. Why?"


Случайно попавший мусор.


:+1: for using the float property.


true


sync


Am I dummy? Are `taskRules` and `taskSchedules` never used?


Is this example complete? Because you need to pass the rules and schedules?


I agree with Paolo that the first part of this change is good (CPSVal <: CPSExp) but the second is bad. Maybe it should be `case class CPSAdd(l: CPSVal, r: CPSVal) extends CPSVal`?

Overall, maybe the whole "encode the notion of CPS in the syntactic structure of terms" is not worth it for the lecture, I don't know. I seem to remember to have added this code mostly because I wanted to teach about the technique of encoding stuff in the syntactic structure of terms, not because I think it is a good fit for CPS specifically.


get rid of this variable, change delta_def function to not assign to the variable. Instead add new function which will compute direction from delta and use locally like this:
var direction = get_direction(delta);


Здесь 'json', а ниже в коде `JSON`.


Change name for better understanding the method intention


No u r genius.  dead code


use descriptive name


незакрытый тег


I mean omx_codecs, not omx_components


There's still an issue with GoParType ...

```go
type foo struct {
	bar string
}

var (
	baz = []*(foo){
		{
			bar: "whatever", // <-- unresolved reference bar
		},
	}
)

```

I got around this with

```patch
diff --git a/src/com/goide/psi/impl/GoPsiImplUtil.java b/src/com/goide/psi/impl/GoPsiImplUtil.java
index 73be6f1..d021cd2 100644
--- a/src/com/goide/psi/impl/GoPsiImplUtil.java
+++ b/src/com/goide/psi/impl/GoPsiImplUtil.java
@@ -599,6 +599,9 @@ public class GoPsiImplUtil {

     if (type instanceof GoPointerType) {
       GoType inner = ((GoPointerType)type).getType();
+      while (inner instanceof GoParType) {
+        inner = ((GoParType) inner).getType();
+      }
       if (inner != null && inner.getTypeReferenceExpression() != null) {
         type = inner.getUnderlyingType();
       }
```

but it seems horrible to me :)


Rename class, please


to jest do przedyskutowania. Ja bym oddał głos na escapowanie przy wyświetlaniu. Filtrowanie przy zapisie do bazy danych jest IMO trudniejsze. Trzeba jakoś to parsować itp. Tylko trzeba pilnować, by to escapować już wszędzie.


Clean up


@smt116 Can i merge this pull request after this correction, and start new PR for tests?
Or stay on this PR?


Fixed


why just not
return !vpwp;


java version?


@neha-ojha i spoke too soon. the object class interface (objclass/objclass.h) includes headers that explicitly assume they are part of the in-tree build. conceptually what we want to do is build a public installable version of objclass.h.


rename delta_def to get_delta?


buzzArr.includes ... returns a boolean
so you can take lines:36-40

```
res.send({'success' : buzzArr.includes(req.body) });
```
which would make better, more concise _code_

it's not complete logic though, since it's still stating pretty much true in every case.
so once you flesh out the post handling, do input validation and any other application logic, you may end up doing something like
```
if ( isValid ) { // your real validation and app logic here
  res.json({ success : true });
} else {
  res.status(400).json({ success : false });
}
```

it is also common, and encouraged practice to construct your response, and responding just once
_depending on how complex your route handling logic gets_
_if it's a simple route handler, do not do this_

```
let status = 200;
let response = { success : true };

if ( isValid ) { // your real validation and app logic here
  //
} else {
  status = 400;
  response.success = false;
}

// respond once
res.send(status).json(response);
```


解决了


You can omit the "auto" here - `immutable x = 5;` works fine as a shortcut to `immutable auto x = 5;`.


Updated the PR with the review comments
Also, started working on SolrSearch batch sink using the SolrOutputFormat class, once completed I will update the PR with those review changes.

And travis  build for this latest commit ‘656f6da’, because of some issues in hydrator-common.
However, build run fine on local machine:
![build_message](https://cloud.githubusercontent.com/assets/13198718/16809439/f93198d2-493d-11e6-823b-d32aeb8b1651.png)


Yup, that looks good @vxbinaca!


I changed the h1 and 1st paragraph


Merge Branch drillsqueak-master in order to prepare for PR merge


Is that the sandbox? :meme:


There are few other things, but you can merge it and fix them later.


Please, read about dependency inversion an refactor code according to it.


Jestem za filtrowaniem przy wyświetlaniu.
To jest to miejsce gdzie ataki XSS są możliwe, więc to jest najwygodniejsze zabezpieczenie. W innym wypadku musimy mieć pewność, że dane wyświetlane, a pochodzące zwykle z różnych źródeł zostały na jakimś etapie przefiltrowane.


+1 Nice.  I like when global variables are made less prominent by introducing them at a higher point in the function call stack.


@thelastpenguin sorry lol


@Crayder  can u compile for the lastest commit?


We'll add it back.

Do you feel like the xivo-callme context should still be defined in /etc/asterisk/extensions_extra.d/callme.conf or could it be moved into the rest of the xivo dialplan (i.e. /usr/share/xivo-config/dialplan/extensions_lib_xxx.conf) ? My point is that I feel like the /etc/asterisk/extensions_extra.d directory should only/mostly contains customization done by administrator.


I mean all of them: codec blobs and component source codes.


It is tested now


What to remove?


thank you!


looks good - no comments


Didn't know those existed that's sweet will do @thelastpenguin


Need renaming


Please, add comments


@jdurgin I think it may be better not to take the mutex. If users want to get an accurate file size, they need to have their own lock or lock the file. Otherwise, even we take the mutex when reading _buffer_size, other threads may modify the value of _buffer_size before user get  file size. Is it right?


I really like my own comment.


The following doc(s) have been deployed to the [1621workfl](https://1621workfl-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/pantheon-workflow](https://1621workfl-static-docs.pantheonsite.io/docs/pantheon-workflow)
- [source/docs/assets/images/test-env.png](https://github.com/pantheon-systems/documentation/commit/6fb62d265bc163842d6b2d55ea284c5deee7cfae/source/docs/assets/images/test-env.png)


I don't think so. A 422 is used if your instruction contained syntactically correct but semantically erroneous information in the body of the request. You might be able to stretch it to make a case for this, but I think it's better to return a 400 with an explanation in the body of the response. `{'success': false, 'message': 'Cannot favourite, recipe is already a favourite.'}`


Is it possible to check emptiness in a more short way?


Not fully happy with this change: there is a set of bills which have doc version of type "Introduced-XYZ", or "Enrolled-XYZ" only. These bills are now lost, while in theory could be kept with the old version (provided, bug is fixed).


Thanks for reporting this bug. Fix is already committed.
Btw which OS & compiler are you using?
Cheers


nope, just before releases are cut


Please set less common variable name


pull latest version


@kiyonori-matsumoto Can you give me an example that demonstrates this bug? I'd like to make sure any bug fixes get merged into the main version so that everyone can benefit. Thank you.


Variable name again


很好的第一步


EMS (macumber) - x86_64-MacOS-10.10-clang: Build Failed

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


Method length is too big


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


=O


Dynamic actors are automatically cleaned up by the game, if I recall correctly. Have you tested this? I have a vague recollection of something bad happening when I tried to call it on an actor ref.


Not sure I get the question regarding this change?


This looks good!


@ashleykasim The problem is v2 introduces "load balancer" as a replacement for vip. So, load balancer v2 service is missleading I think. That's why I thought the overall could be lbaasV2service (to make it clear it encompasses all parts of lbaas) and the specific load balancer service would get the load balancer name. Thoughts? If you think this approach is OK I'll explain more in a comment.


"v" =( Please don't be greedy with simbols


@jdurgin Currently, we don't support sync file range, maybe I could just call Sync().


Thx, the images has been updated


The same with name


#4964-AirLoopHVACUnitaryHeatPumpWaterToAir-with-Cycling-flow-mode-causing-Temp-out-of-Range-warnings (rraustad) - x86_64-MacOS-10.9-clang: OK (2217 of 2224 tests passed, 3 test warnings)

 * 10 tests had: EIO diffs.
 * 6 tests had: ESO big diffs.
 * 5 tests had: Table big diffs.
 * 3 tests had: ERR diffs.
 * 3 tests had: Table small diffs.
 * 4 tests had: ESO small diffs.
 * 3 tests had: MTR big diffs.
 * 1 test had: MTR small diffs.

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


@HimbeersaftLP for the command wouldn't it be $human++ instead of ++$human?


Hmm... why scx15 doesn't have AVC decoder?


OMG you broke the frontend with that really hot fix.


Retrieving the Class Name was fixed, so it allows to show the Push Publishing Dialog Box


這一步如果是我寫，我還不會出現 Distinct,  原因是目前的測試案例仍是三本不同的書。也就是有沒有 Distinct, 在這測試案例底下結果都是一樣的，那麼我就無法辨別我有沒有寫對。

到這一步時，因為已經出現兩次
`Convert.ToInt32(differenceBooks.Sum(x => x.Price) * 折扣);`
如果把原本的折扣當作 * 1.0, 那這一行就出現第三次了，已經可以開始重構了。


Ena?


Date Format was Fixed so The PushPublishActionlet can parse data properly upon execution


Time Format was Fixed so The PushPublishActionlet can parse data properly upon execution


Shared with s8830 codes.


I think this script is fine, congratulations


When I tested this everything seemed to work correctly.
On CsWiki reports about this are constrastant and someone says it doens't happen everytime.
I recall that refstuff had a destroyref function that worked also on actors, and worked well.


ok


@Leemoonsoo I will test it tomorrow and fix it if it's necessary.


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


Add missing Push Publishing Actionlet attributes and read them from the JSONParams object


Np


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1114 of 1114 tests passed, 1 test warnings)

![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-34.65%25-green.svg)


doesnt work for me. propably java version


I just copied the static files from the diamm site (I wanted to concentrate on the other aspects of the site and not worry about the style) and then I realized that I wasn't using any javascript. I shouldn't have done that, I'm also realizing now that there are lots of scss files that I have to sort through because my site is not using them.


Assign Workflow-Related properties to Contentlet in order to execute next Workflow Step, in the same way the Save Content or a Push Publishing task triggered on Website Browser work


why not make it `pub static ref` and just import it in the other file?


Add more details of failure, for troubleshooting purposes


cái này là so sánh như thế nào ảnh Bảo?


```
private double[] distinct = new double[] { 1, 0.95, 0.9, 0.8, 0.75 };
...
 return Convert.ToInt32(differenceBooks.Sum(x => x.Price) * distinct[differenceBooks.Count - 1]);
```
我會建議用 dictionary<int, double> 來紀錄「幾本不同的書，擁有的折扣」，而不是透過 array 的 index 搭配弔詭的 differenceBooks.Count - 1。

語意跟 robustness 都會好很多


jjkhkhjkjkj


nice fix m8


@neha-ojha have a look at the `objclass-dev2` branch. I created a file called `objclass/objclass-public.h`. This can serve as a public facing interface. We need to add things to this public interface to get our object classes to build, but we want to be careful about what we add (ie only stuff that we want to expose outside the tree). we might find things that need to exposed, though.


ah I see, that makes sense. In that case Flush() should use aio_append() like you're doing, and call write_completion->release() before returning, since we don't need to do anything else when it completes.


đây là so sánh, đã ghi bên trên ``` {{#ifCond this.slug ../this.slug}}```


The following doc(s) have been deployed to the [1629wpmut](https://1629wpmut-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/composer-drupal-8](https://1629wpmut-static-docs.pantheonsite.io/docs/composer-drupal-8)
- [docs/create-wordpress-site-networks](https://1629wpmut-static-docs.pantheonsite.io/docs/create-wordpress-site-networks)
- [docs/managing-wordpress-site-networks](https://1629wpmut-static-docs.pantheonsite.io/docs/managing-wordpress-site-networks)
- [docs/migrate-wordpress-site-networks](https://1629wpmut-static-docs.pantheonsite.io/docs/migrate-wordpress-site-networks)
- [docs/wordpress-site-networks](https://1629wpmut-static-docs.pantheonsite.io/docs/wordpress-site-networks)
- [source/docs/tags.html](https://github.com/pantheon-systems/documentation/commit/3d0cce7a67cd574e354251846fa6627dc8fa33c8/source/docs/tags.html)
- [source/docs/tags/tag.html](https://github.com/pantheon-systems/documentation/commit/3d0cce7a67cd574e354251846fa6627dc8fa33c8/source/docs/tags/tag.html)


At first common best practices should be familiarized, than design and architect patterns. You can find them in materials I've sent you. Then code refactoring and second check. For now it's hard to check the algorithm itself.
Feel free to contact me as I'm going to refresh these theme too =)


Is this one more appropriate?

>422 Unprocessable Entity (WebDAV; RFC 4918)
The request was well-formed but was unable to be followed due to semantic errors

the semantic error being that the recipe is already a favourite


如果這一行只是要用Select(),  那麼上一行`var differenceBooks = books.Distinct(new BookCompare()).ToList();` 就無須 ToList() 了，是沒有必要的浪費。

另外 `differenceBooks.Select(x => books.Remove(x)).ToList();` 這行如果只是為了執行迴圈，建議直接使用 List<T> 的 ForEach()，因為這一行沒有 assign 給任何一個變數，這是很奇怪的程式碼。