Hmm, I can't find that conversation, but at this point I think this change should be reverted as our scripts in other locations have already been updated use the new HistBin naming. I had just submitted an update to the abbreviations in openbr as well (did not see this before that). Please let me know if there are issues with a revert, otherwise I will submit a PR.


one should set ```node => false,```, so it is clear, that this shouldn't be a node.


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1123 of 1123 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.56%25-green.svg)


Thanks, @swannodette .


I usually drop my ip addresses from the test.


ditto


Ah yes makes sense. Currently windows includes ruby in `puppetbindir`, you have to scroll far to the right to see it.


There's a tiny inconsistency in formatting: "cp    $FILECAP" vs "cp $FILECAP".


does not numberic original port? :-)


Phil, can you describe what the ost-tag-* designates?  Just to clarify why the 5 tags below are included.  Some are assessment-only, but TEKS and LOs appear on other kinds of items.


polar express


#30 where does 'ubuntu-14-04-x64' get resolved in this instance... the only way I've found to resolve to images of my choosing is to use the image id in this field.


this commit breaks orm behavior in my app
had to revert to 2.3.11 couse some date ralated things stopped working
will debug to see what is wrong...


where will we use this tag?


слишком большое дельта. необходимо установить значение в 1e-10, не меньше.


Like ListViews on Android, `UITableView` has support for recycling cells. You generally want to use `dequeueReusableCellWithIdentifier` rather than creating a new cell directly since this reuses any cells that have been recycled (i.e., no longer visible).

Might not be a big deal here since we're only showing a couple rows, but I don't see any reason *not* to.


For common images provided by DigitalOcean, we provide slugs (such as `ubuntu-14-04-x64`). For your own private images, you must use the ID (There's not currently a way to tag your private images with a slug).


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1126 of 1126 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)


In acls-1.2.rng, the "attribute" is not for matching xml nodes.  It's actually for controlling the access to the specific attribute instead of the entire xml node. That's the reason why it can be combined with any of the others.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2208](http://teamcity.monogame.net/viewLog.html?buildId=5617&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 441, ignored: 6 Build time: 00:13:57


`return this;` on separate line for consistency (and better style)


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/913


So we'd have duplication there.  Should it continue to?  Or should that be moved into a proposed/renamed systembindir?  I'm not familiar with the history of why puppetbindir is the way it is on Windows.


Probably: I am now able to run a 3-year data import, day by day, without any manual intervention (all notifications are processed before moving to the next)! :sunglasses:


maybe keep git happy by putting a NL in


Hi guzza,

What exceptions are you getting? Could be some lingering calls. Also, what version of MySQL? Guessing 5.x running on Linux? Or on Windows with case sensitivity enabled?


Make this `exports.validate.parsed = require('./validate/parsed')`. That way the parsed interface can be used like:

```
var validate = require('mapbox-gl-style-lint').validate;
validate.parsed({...});
```


Hi, yes :)
> Le 3 mars 2015 à 17:02, Jocelyn Fournier <notifications@github.com> a écrit :
>
> Hi,
>
> Could you tell me if you have an index "product_manufacturer" on the table ps_product ?
>
> Thanks and regards,
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/PrestaShop/PrestaShop/commit/27e4841246ad0df00b65d6d3fcbadc50576db18d#commitcomment-10010059>.
>


or you could add a .gitconfig, I think, that makes .heapsnapshot files be binary... but I don't think its worth that much effort


@erikrose Sorry, but I'm getting various errors with your setup (the fabric command didn't at all).

```
/Users/jezdez/Code/git/django-tidings/docs/views.rst:9: WARNING: autodoc: failed to import function u'unsubscribe' from module u'tidings.views'; the following exception was raised:
Traceback (most recent call last):
  File "/Users/jezdez/.virtualenvs/tmp-30eb1ce9cb02f2dd/lib/python2.7/site-packages/sphinx/ext/autodoc.py", line 335, in import_object
    __import__(self.modname)
  File "/Users/jezdez/Code/git/django-tidings/tidings/views.py", line 3, in <module>
    from tidings.models import Watch
  File "/Users/jezdez/Code/git/django-tidings/tidings/models.py", line 111, in <module>
    class NotificationsMixin(models.Model):
  File "/Users/jezdez/Code/git/django-tidings/tidings/models.py", line 118, in NotificationsMixin
    Watch, related_name='%(app_label)s_%(class)s_watches')
  File "/Users/jezdez/.virtualenvs/tmp-30eb1ce9cb02f2dd/lib/python2.7/site-packages/django/contrib/contenttypes/fields.py", line 266, in __init__
    from_fields=[self.object_id_field_name], **kwargs)
  File "/Users/jezdez/.virtualenvs/tmp-30eb1ce9cb02f2dd/lib/python2.7/site-packages/django/db/models/fields/related.py", line 1322, in __init__
    super(ForeignObject, self).__init__(**kwargs)
TypeError: __init__() got an unexpected keyword argument 'related_name'

```


QwtUpdate (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


That's with Django 1.7.x


No strong opinions, just want to stay consistent :smile:.


Ack, fair enough. I was just afraid of too much logging.


only for use on Watch Physics?


saw approval from SimonSapin
at https://github.com/servo/servo/commit/4c8bde5736853ae68332d1f97cc6f0b02499e35f


merging servo/servo/counters = 4c8bde57 into auto


servo/servo/counters = 4c8bde57 merged ok, testing candidate = 5cd6316a


Ez így nem lesz jó!
Ha a bejövő szám 33???


I'm attempting to make our lives easier with some helper methods, but I'm running into problems with pushing said changes. Here's to hoping we don't have the Second Great Niaderp while I fix these errors.


do we use this any any instance other than snap-lab?


Hello

My company has signed yet a CLA.

Best Regards
Franck CHRISTIN | Phone: : +33 (0)1 70 38 51 45 | Fax +33 (0)1 70 38 51 51 | Email : franckc@bluestreaktech.com<mailto:ericg@bluestreaktech.com>

De : Thom Dedecko [mailto:notifications@github.com]
Envoyé : mardi 3 mars 2015 18:33
À : franckchristin/js_mse_eme
Cc : Franck Christin
Objet : Re: [js_mse_eme] Endurance Test 1 and 2 failed due to a change from sb.append to sb.appendBuffer in the function createOneShortTest (4054a36)

Thanks for contributing. I can't find a signed Contributor License Agreement for your username. I can't accept contributions unless we have one on file. It should be pretty quick to do. Go to the link listed in this file: https://github.com/youtube/js_mse_eme/blob/master/CONTRIBUTING

—
Reply to this email directly or view it on GitHub<https://github.com/franckchristin/js_mse_eme/commit/4054a3644d16d3278bb1835a81e1cd9e0e12d387#commitcomment-10012084>.


use for Fun in Physics, Work in Physics, Boundless Physics?


You added your stylesheet reference properly, but you should have also removed this unnecessary `link` tag as well. It's not breaking anything, but it's superfluous.


It is, but this is for user responses on how the call went.  I'll change the constant name to make that more clear.


It van egy kis gond. Mi van ha a bejövő szám osztható 3-al vagy 5-el vagy 7-el és tartalmazz is az osztót. Pl: 33, 55, 77.


I don't understand this change. It just down-cases a constructor. I'm missing something.


Remove the leading slash in your `background-image` URL and this background will pop into place. I like the background image you've chosen, and it does help bring the page together.


No, we will remote NotTransactional. See https://issues.openmrs.org/browse/TRUNK-4606


What does `cs` stand for? I'd suggest something more verbose since I can't figure it out!


Yep, that's exactly what I had in mind. I was worried about values it doesn't understand (i.e. `AE_QUALITY_GPU`), but it just seems to ignore these values and for this specific case it's initialized with `AE_QUALITY_UNKNOWN`.
I will test this tonight or tomorrow (waiting for a fix upstream).


Great, thanks!


In acls-1.2.rng, we allow "tag" combines "ref", because we'd want to differentiate objects like <primitve id="rsc1"/> and <lrm_resource id="rsc1">


oh, I was just referring to the string 'hungup' -- that is, I'm not breaking anything by changing it to 'hung up', am I?


Sorry, the ruby path should be moved to be consistent with unix. Adding it to puppetbindir was a short term measure because we didn't have MSIs in the AIO pipeline yet.


I don't think <abstract> exists in cnxml (http://cnx.org/eip-help/tags).  Currently the key terms are in a table near the top of the content.  They do have the class "section-key-terms" although I can change it to "key-terms".  If Tutor will use this, should it be os-key-terms (the css uses it to style the key terms table)?


Variable names read funny a bit ? :)


This could have been separate commit. But doesn't matter.


Ignore this file.


r+


[SideCI/RailsBestPractices] move model logic into model (@documents use_count > 4)
reference: http://rails-bestpractices.com/posts/2010/07/21/move-model-logic-into-the-model/


This URL format is defined by `request`, is it?


@overminder 不是我干的。。。。github自动生成的这条


If so, could use a comment: convert from our CLI format to the format request wants.


These `transition` properties are not quite right. They should specify the property you want to transition and the time you want the transition to take. Right now, if you inspect the H1 in our developer tools in the browser you'll see that these properties give errors. Here's what it should look like:

`transition: width 2s;`

That will give you the smooth growth you want. However, you should probably also make sure that the default width is enough to not cause the text to wrap. You can solve that in two ways: Either you can make the width larger than it is now (greater than 8em) or you could use the `white-space` property:

`white-space: nowrap;`

That's the most reliable way to make sure content does not wrap onto multiple lines.


This code has wrong indentation.


QwtUpdate (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yeah, but I don't care.  I'll expand it.


@guanlun Computer Vision的第一的project


Code Review 12: Testing code review

Can anyone help me resolve this code? It should print hello, world, but it doesn't.

Hey, could someone look through this and give me some feedback conserning this?
Sincerely @meling

---------
@meling, follow this tread for feedback.


I'd put the entire URL in, people can see what the protocol is, and the rest of the URL at the same time.


You should inherit from exception.Invalid (this is a bad request) and set the msg_fmt so that we can return a proper error to the client.


should this be:
problem
para
a class="os-embed" href="http://exercises.openstax.org/get-exercise-by-shortcode/k12phys-ch12-ex001" <-- singleton (closed) link
?


Ez így nem lesz jó!

Mi van ha a bejövő szám 33?


No problem! The tests help us in these situations, hehe.


blah blah blah...


ah, good!  I'm leaving a few comments.

Spontaneously though, I think the best fix would to be find the place where it calls `verify_fingerprint` with the "wrong" fingerprint. There, it should already know that it scanned smth prefixed with "OPENPGP4FPR:".


Oh, maybe this too simple. Just another test.


Nem lesz jó!,ha a bejövő szám 55.


This comment is no longer true, and the return statement should be removed.


I would expect that the loop creating the :opaques option sees the same opaque more than once. However, I would have expected this option to be passed as a Set, which ensures uniqueness


77 esetében nem lesz jó.


Was this code broken before? Service to ServerService doesn't seem related to unix, and I thought we renamed ServerService to Service?


What does `unwrapkB` mean? Should this be `unwrapBKey`?


Blarglargl.


Where you able to get this integrated?


novo novo


Seems to have a lot of unrelated changes, I'm a bit confused.


Hey @gowong, nice fork. I have intentions to make the Font Awesome dependency a config option going forward. Thanks for the idea. https://github.com/Blooie/ember-cli-notifications/issues/8


It's good to explain what the code is supposed to be doing. But it's even better to tell why it does it.  So I'm asking (naively): Why do you strip the prefix.  I know the answer, but someone else is likely to not know, so it'd be good to leave an explaining comment.


Fair ... this one indeed looks very dubious


Option 4 makes sense to me, plus perhaps we should drop the `systembindir` setting? It looks like it was added for AIO testing but isn't used in AIO testing so might be a safe removal. /cc @kevpl


commit message spelling/grammar:

grammar: customization

"service manager" vs "service-manager"


Yes, I agreed. Removed it in later commit.


a minor nitpick: Wouldn't `startswith` be a bit cleaner? Also, let's try to not repeat ourselves. The "12" is hardcoded twice. It'd probably be better to have this information stored only once.


Te recomiendo que elimines la carpeta .ipynb_checkpoints del control de versiones. Cambia a cada rato y no aporta nada: solo es un control para que tú no pierdas cambios localmente.

No sé cómo se hace en Windows, pero desde la línea de comandos sería así:

```
git rm --cached .ipynb_checkpoints
```

Luego, puedes añadir la línea `.ipynb_checkpoints` al archivo .gitignore para que git ignore esa carpeta en adelante.


This is a mismatch between the Android code and the upstream JSON.  Android
used `kA` and `kB`, hence `unwrapkB` (which, IIRC, matches warner's docs).
I could change to `unwrapBKey` throughout.

On Tue, Mar 3, 2015 at 9:52 AM, Brian Nicholson <notifications@github.com>
wrote:

> What does unwrapkB mean? Should this be unwrapBKey?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ncalexan/firefox-ios/commit/d943f5c3e9c47ab347c57896ffc18714992921b6#commitcomment-10012477>
> .
>


The documentation probably needs to be updated, too, then.


I like the spinning, and the styling is very attractive -- the stack of cards look works here. However, the spin takes a long time, and during those seconds I can't click on the links very well. I think the best solution would be to cut the spin time--probably in half. 4-6 seconds is a long time to wait before you can click a link...


This looks good.


No source?


The problem is not reloading the config but changing log levels by commands.


This probably works just fine.


Think we can remove solution here -- they will all be TS solutions which the XML teams will ignore.


For Canvas variables, we seem to have a mix of documentation strategies (e.g. https://github.com/ets-berkeley-edu/collabosphere/pull/3/files#diff-89a0af99d63d5802998daba843768dbdR22) that prefix the variable with or without `Canvas`. We should make these consistent.


excellent :)


Great use of the `nth-child` selector and kudos for using such specific selectors. In a design like this, we can afford to be very precise with these sorts of definitions. Kudos to you!


Also, the description should be `The domain on which canvas is running` per the DB model


does this matter? do you expect people to actually inherit from ServiceManager? I thought it existed more as a form of documentation. Though I guess, if it's expected to be an EE, then the sample should be an EE, too.

What are events intended to be used for? The entire class is basically an alternate form of EE, so now that it is also an EE, I'm not sure what notifications are intended to be events, and what notifications are intended to use the "virtual/abstract functions".


`The basic LTI key that will be used to embed the tools into Canvas` per DB model


Ezt nyugodtan kitörölhetted volna.


So much for polymorphism :/


No me había dado cuenta de que existía, lo añado!


I think that falls under "Actually Writing a Test" since the test-infrastructure is explained there^^


This is an odd construction.  We know the loop length, right?  This looks like refactored code.  Why not just
```
for (int i = 0; i < items.length - 1; i++) {
...
}
builder.append(items[items.length - 1])
```


I'm glad this is getting more use already! :D


Hey Ryan,

This is very cool - not being able to do this has definitely been an initial source of confusion for some clients.

The only weird thing I am seeing is that once you upload a new image, the grid of thumbnails no longer includes images from repeater fields within the page. They are shown when you first click the CkEditor image button, but are gone once a new image has been uploaded.

Also, just noticed the wording of the "Select Another Image" button. This sounds to me like you can select more than one image - "Another" suggests one more. Perhaps "Select Different Image" or even just "Select Other Image".


Set for home PC


Can we make this `The requested canvas instance` ?


@jacobwvandijk See this on how to configure your path properly http://stackoverflow.com/questions/3701646/how-to-add-to-the-pythonpath-in-windows-7


@xni PUT is used to create a resource, this particular method is used to update existing resources.


Sorry I copied what you sent me. v0.12.0 is what I get.
ᐧ

Thanks
David

Shaping Connected Experiences

<http://davidmermelstein.dunked.com/>  Portfolio
<http://davidmermelstein.com/>     <http://www.linkedin.com/in/merms>
Linkedin <http://www.linkedin.com/in/merms>    Twitter
<http://www.twitter.com/dmerms>      773.744.0760

On Tue, Mar 3, 2015 at 11:33 AM, Martin N. <notifications@github.com> wrote:

> Just "node", a space, "--version". That should print out the node.js
> version you're using.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AVGP/susi/commit/fa59094a26641840530c9fef5110fa1f75d7b7e0#commitcomment-10012101>
> .
>


Why do you need `deepCopy`?  It's not very idiomatic in Java.


yeah, just wanted to mention it's missing in the document.


nit: most comments are capitalized and have periods, the ones below are like that.


I think this should be a `<` instead of `!==`, since a TODO that passes should increment count but that should probably not be a failure overall.

I suppose whatever is listening to the consumer's events is able to look at the result and interpret them their self. Then that raises the question of whether Consumer should be trying to do this at all. Perhaps it is something the Test class should be doing instead?


Látszik a kódodon, hogy C-ben kódoltál.
Jellemzően nem így szoktuk formázni a kódot. Használd az ALT+SHIFT+F bill. kombót, az beformázza.

Ennek az az előnye, hogy mindenki szeme ahhoz szokott hozzá, s könnyen tudja olvasni. Egységes.

egyébként a kód ellátja a feladatát.


this fix might not be enough, I hit a similar issue and had to use the profile repository to resolve all repos. See https://github.com/grails/grails-core/commit/f69648dc7a1d6a088f539934766fd8fa1c2fb1b2#diff-3ae0132d3c9fdf83a3458da1de22d1d1L241


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (629 of 629 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-3.96%25-green.svg)


is Virtual Physics a feature?


This already was null-safe, right?


> The same problems here, solved by using QT_NO_KEYWORDS and some sed
> magic ;-)

how can you teach sed to distinguish between the pseudo-keyword and
string literals with the substring "foreach"?

> The culprit is "namespace foreach" in foreach.cpp. This will be
> clobbered by the preprocessor if the Qt "foreach" macro is active. The
> same happens in Boost.Signals with the "signals" macro.

therefore push_macro / pop_macro.

> Blame Qt for introducing "pseudo keywords".

or blame the c++ language for having a preprocessor. or objective-c for
having a nil keyword/define which used to clash with fusion's nil class.

but blaming does not help solving real-world compile errors.


Can we make this

```
if (err) {
  log().error({'err': err, 'id': id}, 'Failed to get a canvas instance');
  return callback({'code': 500, 'msg': err.message});
} else if (! canvas) {
  log().debug({'id': id}, 'A canvas instance with the specified api domain and consumer lti key could not be found');
  return callback({'code': 404, 'msg': 'A canvas instance with the specified api domain and consumer lti key could not be found'});
}
```


Erre miért is volt szükség? Majd nézzük meg órán.


per yesterday's discussion, are we making these notes?


```ruby
@comment = @post.comments.new(comment_params.merge(user: current_user))
```


Itt is a formázás hiánya. Nehéz így olvasni.


a void-ra a metódus argumentumban nincs szükség.


Is it worth having these members not show up instead of their values just being `0`? Would simplify this code and also make the emitted objects more uniform in shape. I imagine anyone using this code is going to do `if (final.fail) {}` instead of `if ('final' in fail) {}` anyway, so simpler internals are probably a net gain.


I'm still not happy that we're mixing uppercase constants when we (and iOS) use CamelCase everywhere else…


It's safer otherwise potentially a resource leak


You can't edit code-review main post


No, not really since it will try the next host potentially.


Приведённый `'/'.join(…)` не подходит для случаев, когда куски кончаются слешем. По хорошему надо объединить libskygrid с libscheduler и заюзать тут этот класс: https://github.com/anaderi/skygrid/blob/master/libskygrid/libskygrid/common.py#L8

Предварит


Yah, sorry. I've corrected that on Github, because my phone inserted spaces
that shouldn't have been there.

When it comes to that, stick to github rather than email content, as I can
update that but not the email.

Could you try installing/using 0.10?
What makes me wonder is that also other module installations (we've tried
"npm install -g grunt-cli" didn't we?) also fail..

Which operating system are you on? Mac? Windows? Linux?
Am 03.03.2015 19:00 schrieb "damerms" <notifications@github.com>:

> Sorry I copied what you sent me. v0.12.0 is what I get.
> ᐧ
>
> Thanks
> David
>
> Shaping Connected Experiences
>
> <http://davidmermelstein.dunked.com/> Portfolio
> <http://davidmermelstein.com/> <http://www.linkedin.com/in/merms>
> Linkedin <http://www.linkedin.com/in/merms> Twitter
> <http://www.twitter.com/dmerms> 773.744.0760
>
> On Tue, Mar 3, 2015 at 11:33 AM, Martin N. <notifications@github.com>
> wrote:
>
> > Just "node", a space, "--version". That should print out the node.js
> > version you're using.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/AVGP/susi/commit/fa59094a26641840530c9fef5110fa1f75d7b7e0#commitcomment-10012101
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AVGP/susi/commit/fa59094a26641840530c9fef5110fa1f75d7b7e0#commitcomment-10012648>
> .
>


This seems interesting. Infinite looping and break with value. (New for me)

What I would have done in an old fashioned way is do a recursive loop and break if no record found.

The one you did is nice. :+1:


You may want to check if the list is empty also. I think that if you apply `forall` on an empty list, it should return true


If I haven't said it yet, this looks ready to go, and I'm itching to see it work. Could you commit it to master (if you haven't)??

Thanks.


Yeah! Though I think we may have done some of the same changes, make sure to rebase onto master.


Miért van itt egy for???


Yuck, this all feels so fragile.


r+


Yes, that's the parameter from the json


Cool, looks great!
But it would be better if you looked into python [generator expressions](https://docs.python.org/2/reference/expressions.html#generator-expressions). We don't just want them to be iterators rather than lists - the only reason for that is that they can be lazy loaded. So, how about something like

```python
location_cursors = (query_tweets(api, query=q, limit=l) for q in queries)
```


Agreed. This fixes the issue I was having, we'll see if other cases break
it.

On Tue, Mar 3, 2015, 08:16 Jake Bolewski <notifications@github.com> wrote:

> @staticfloat <https://github.com/staticfloat> I changed the logic in
> 34a78d4
> <https://github.com/JuliaLang/julia/commit/34a78d4640ea29abf34f7e9cdcf39b544b515237>
> based on this discussion, I think we should try to test this if possible as
> our coverage for error code paths is really bad.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaLang/julia/commit/25b89e8470ea6380bac72e9720d32c939fbdc673#commitcomment-10010396>
> .
>


Látom később kivetted, jól van.


Remember, we're actually posting to *our* /compile path. In other words, send a post request to blockly.codebender.cc/compile and put the JSON into the data or body of the request.
I think you'll want to specify the URL as a relative one rather than an absolute one.


Declaration on header file.


Great idea.


I don't see anything in this file being used anywhere else in this PR. Does this belong here?


@temasek Due to the settings re-organize commit, your id should be @id/display_and_lights_settings if I'm not mistaken.


:many thumbs up:


I return buffer at the end of file for warning.


Issue #34: change default `database` to `default-filestore`


`Get a course. If the course doesn't exist yet, it will be created.`


Jó lenne, ha rá tudnák jönni, hogyan kéne mergelni a 2 branchet... Én s te mást látunk.


If the cursor is at the end of a commented line and I push `[Shift] + [Pos1]`, ZendStudio marks only the contents of the comment, but no the comment signs itself. I simply did not properly wiped it out ;)


You can completely delete this file... we do know where to find it if we need it again, after all.


Do you know the invokation parameters? from `assertIn()` and `assertIsInstance()`?


Lol.  Lipstick on a pig...

If you're wanting an easy, low-risk palliative at least just put an exception handler round everything in the thread lambda in the window class.  (Basically all my PR #258 did in the first commit).  Running a secondary thread without that exception handler is just asking for trouble anyway.


`The requested or created course`


@bklare sorry, I thought this was a quick solution for Scott's issue I didn't realize you were using histbin. From this point forward, I will consider the reorganization changes as stable and submit PRs for any changes I make.


FYI see marked up example from robust sample content from W&N: "This video contrasts the way we thought about motion and force in the time before Galileo’s concept of inertia and Newton’s first law of motion with the way we understand force and motion now. Refer to [Link to/ Figure 04_02_wagon] as you watch.
[https://www.khanacademy.org/science/physics/forces-newtons-laws/newtons-laws-of-motion/v/newton-s-1st-law-of-motion]"


Note that this isn't really a failure of polymorphism. The reason for this utility is that for longs we can avoid using `IN (?, ?, ?)` with `selectionArgs`, because it's insane to build a new array of the string representations of some numbers, pass them to sqlite, and then have sqlite reconstruct the query.

Instead we can save some of our precious 99 `?` slots, some efficiency, and just build the `IN` clause ourselves.


@davidism is right, see this answer for explanation: http://stackoverflow.com/questions/2390230/do-copyright-dates-need-to-be-updated/2391555#2391555


Megvan, kell egy Pull Request, anélkül nem tudom mergelni.


Oh, right.  I think all the uppercase constants are new to this patch series; I'll land with camel case.  (I landed the last series with camel case.)


What was the motivation to enforce uniquness of values? It seems like in C only Enum names need to be unique.

I am wrapping some low level I/O (mraa) for the Intel Edison and multiple names can refer to the same pin on some boards.


Why do we need the `canvas_canvas` here?


Wow. So in an opposite castled situation with kings on b1 and g8, we weight pawns by only half as much? Really interesting. What gave you the idea?


1540218-SlinkyGroundHX (mmatts) - 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)


Should be `return callback({'code': 500, 'msg': err.message});`


Namespace here is a compulsory.


This only happens on 8.1.2 and 8.1.3, correct?  Maybe that changed something, since I haven't tested on 8.1.3 (I don't think I was on 8.1.2 either).  From the information you gave me, I can only see that WIRListingKey is missing the <key> field, which causes the error, but I have no idea why that isn't being populated.


I need to test use of iframe tag; not included here: http://cnx.org/eip-help/tags


I wanted to make it clear that it is tunnelling http requests over an ssh connection and _not_ speaking a new ssh based pmctl protocol. When I looked around `http+ssh` seemed more common than `ssh+http` for what I was doing.

This also keeps our options open if we later decide to implement a more native ssh based protocol, which is something I'd like to attempt in the future.


Return buffer at the end of function avoids warning.


Declaration of specialization in header file.


What happens when `items.length` is zero? There has to be a check, whether before or after. This code has been noodled on a lot :)


f


don't you want to link to official doc?

2015-03-03 19:09 GMT+01:00 Makman2 <notifications@github.com>:

> Do you know the invokation parameters? from assertIn() and
> assertIsInstance()?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/51a14458918e116fee84f0a7a49f2880b5d8ecf4#commitcomment-10012898>
> .
>


Forkoltál? Talán az a baj.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/914
success: http://build.servo.org/builders/linux2/builds/912
success: http://build.servo.org/builders/android/builds/950
success: http://build.servo.org/builders/mac1/builds/1129
success: http://build.servo.org/builders/mac2/builds/1129
success: http://build.servo.org/builders/gonk/builds/100


@philschatz could you add the teacher demonstration class attribute here?  content goes in a new note within the TS note.


Why inheritance? Why not just abstract out that one minor thing in reflection?


It takes a view model with observables and turns it into pure JS, so the debug view can render it


Looks good to me.  I'll check it out and run it myself.


cc/ @josevalim


Yeah but a note is nice so people can directly see how to invoke them.
But I already found them, the search function GitHub is quite nice for such problems :)


Essentially because we don't have persistent data structures. We end up passing `ExtendedJSONObject`s around, using them to compose records, which can then be mutated. In Clojure we wouldn't have to do this, but in Java we either suffer through surprising bugs due to reference sharing, or we're careful to copy our data structures.

Rather than implement a shallow copy and trust that we'll never, e.g., add an array of tags, I opted to just bite the bullet and implement something correct in the first place.


Return buffer at file end avoids warnings but it's less effective.


@noopkat brill


Wow! Kíváncsi vagyok miért kellet minden fáj tartalmát teljesen kicserélned.


How would you do that with the 2 compiles? You want one called if 2.10, the other called if 2.11 and the class names used must be the same


Namespace here is a compulsory.


This probably wants to be installed as the buildslave user. I believe plugins are per-user.


https://docs.python.org/3/library/unittest.html#assert-methods

2015-03-03 19:17 GMT+01:00 Makman2 <notifications@github.com>:

> Yeah but a note is nice so people can directly see how to invoke them.
> But I already found them, the search function GitHub is quite nice for
> such problems :)
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/51a14458918e116fee84f0a7a49f2880b5d8ecf4#commitcomment-10013062>
> .
>


fast-forwarding master to auto = 5cd6316a


FYI - There are no examples of Check your Understanding" in the robust sample content.


Why not just make the request? It will fail if the socket doesn't exist, and the failure can be reported.


@jfols fyi - removed what I had originally added since it didn't work correctly


Forkoltam..de mi a baj?


Ennek nincs itt helye. Inkább egy main osztályt kellet volna létrehoznod.

A FizzBuzz-er egy olyan osztály lehetne, amelyet mások felhasználnak, akár többször is egy nagy projektből. Persze így is megtehetik, de kicsit csúnyácska , hogy van benne egy main metódus, amit soha senki nem fog használni.

Órán majd beszéljünk róla. (Szóljál)


there is also boost::flat_set that is very fast, maybe there are some other interesting container in boost::container as well.


Can you list the accepted tool ids?


Yes; this was left over from a previous fix. I undid this part of the change and added a test.


You can remove the parentheses around 1000.


Do you know why these permissions are needed?  I can't find where unix_chkpwd writes to /tmp from looking at the source upstream.

If writing to /tmp is expected, a tmp_filetrans() would be a better solution than writing to tmp_t.


Nem ugyanazt látjuk, s nem tudom, hogy miért. Majd utánna nézek, ha akarod látni mi van az enyémen klikk a nevemre s onnan lépj be a KovaRozs ra.


I did abstract the one minor thing but for each compile so I'm not understanding what you propose?


saw approval from pcwalton
at https://github.com/bjwbell/servo/commit/7855d0b2322817c883235369d77b553e47ecec72


merging bjwbell/servo/inline_block_overflow_ref_test = 7855d0b2 into auto


bjwbell/servo/inline_block_overflow_ref_test = 7855d0b2 merged ok, testing candidate = e65a5452


That'll probably work. Might even be able to get rid of the `@unix_only`, unless `image_file` works much differently on Windows... which it might.


Yes, that's one of the reasons why UNIX sockets are nasty. You may want to put this one in ``/tmp``.


right, added to the commit message


Ezt törölhetted volna, vagy helyettesíthetted volna mással.


Other than that the logger is disposed first, this is already being done by the destruction of  `using (Disposable.Create(() => animatedGifWindowToken.Cancel()))`, so I don't think it buys much.


Nice thanks :)


Interesting point. In our case, one of the names needs to be canonical for the sake of printing. Extra aliases can be added with `const` assignments, so AFAICT allowing duplicates within an enum definition is not strictly necessary.


Shouldn't the launch URL be different? It looks like this POST will end up in a redirect to the same URL


You are correct, this was an accidental include.  Good catch!


Didn't notice that I'd even had to make those kind of changes. Will look at that on wednesday.


Oops. Now it's double cancelled


Rather than passing in the title and description, we could also look them up in the `getBasicLTICartridge` function using the `ID` property?


Well, if I merge it, we should not mention this like that.
Please, just add a contributors section (also, could you list others contributors)


No need to change this though


The ELSE is not needed, since you return in the IF part


Phil this should be updated (worked example)


It should be better to have other versions of Django, no? Like 1.6 ? May be the tests pass.


worked examples


The same here, if I merge your PR, no need to install from your repo.


This is not conventionnal, template tag are always at the top of the template no?


Same comment here


Ennek nem itt a helye. Külön main osztály lett volna jó.


@mciruzzi Tincho, no se si estas usando este metodo desde el cliente, pero tene en cuenta que en este PARTICIPANT_ADDED te estaria llegando un array con los nombres de las personas que agregaste


yeah, I like this change.
changed and rebased


What's the difference between `app` and `server`? I thought they were synonyms. In which case these would be `_app` and `_meshApp`.

But perhaps, `express()` should be `this._server`, and the `mesh server/app` should be `this._app`? That would cause less churn down below.


@dsjen This can be simplified further. See https://github.com/edx/edx-analytics-dashboard/commit/c9b7077d39fa2bfb33ad6f54854ef741096344cb.


...welches Deine Art zu entwickeln buchstäblich verändert.

Oder?


Why do we need an id ?


when does this evaluate to false?


why not put it into an own test?


***No* german Comments!**


megneztem.. vagany


I agree with adding the newline since the file isn't actually a valid heapsnapshot.

If it was replaced with a minimal valid heapsnapshot file, and it is binary or pseudo-binary, you could add a `.gitattributes` file in the repo top level containing:
```
*.heapsnapshot -text
```


Should follow the guideline above


`test_default_coafile_parsing` or so


I'd prefer if we didn't skip the last bit of this test -- this implementation should catch a string with the wrong encoding just like the other one does.


Well, I made some comments.
Very happy to have your feedback.


In this case, all the check must be removed on database. For me I prefer to have this check, it's my opinion.


@openstaxalina Check Your Understanding is the element formerly known as Formative Assessment.
@philschatz Yes there can. See example content below. Check Your Understanding should appear only before the group of questions (not before each exercise).

[Formative Assessment]
1. What is Newton’s third law of motion? [S4-LO1] [DOK 1] [blooms-2]  [S]
How does it compare with the other laws?
What is this law used for?
Give an example problem that uses this law.
2. Considering Newton’s third law, why don’t two equal and opposite forces cancel out each other? [S4-LO1] [DOK 2] [blooms-4] [S]
Table 8: Formative assessment testing table
...


Needs docs


some picky notes to this script:

9: maybe use mktemp
13: check if SRC is dir and exists before removal (not sure how rm -rf / on osx will behave, imagine a space)
14-18 (chain with &&)
20: cd $SRC && ...
21: find . is not nice (imagine SRC is / or something else, then you get cd / ; find . ) will probably make a big
archive


user_t, logadm_t, and staff_t can transition to nirtcfg_t.  files_manage_boot_files() contains permissions that you would probably like to restrict to only privileged users.


`[{pk, last_insert}]`. The current code will convert the current pk to a string and then it back to an atom. :)


Your commits 're getting better!


Same here


`if the user doesn't already exist`


Ok just submitted revert. Aside from this being the path of least resistance to getting things working again, I also think this HistBin name is more informative.

https://github.com/biometrics/openbr/pull/341


Get this question answered by the user when you're determining server information information.  Once we get to this point, our server connection should work properly.


I feel like this should read:

> In order to indicate **this** to the client this extension should be implemented.


damn **sou**...


If the user already exists, should this function also take care of updating the user's role, name and image in case they have changed?


@openstaxalina Virtual Physics are the simulations. @kathi-fletcher I just realized the Robust Content does not have links to the sims. When do we get these links?


Should go below `pg` and `pg-hstore`


The corresponding strong-pm change makes PM's lib/server.js Server class inherit from ServiceManager, so subclassing is the intention.

@kraman is the ultimate purpose of the EE so we can distinguish between events and hooks?


You need a mingw-w64 cross-compiler. Technically clang has cross-compilation capabilities built-in due to the way LLVM is designed, but it's not exposed in an easy-to-use way. I don't know whether there is a reliably binary installation method for mingw-w64 compilers on Mac. On various Linux distributions it's pretty easy and I can point you to the packages you need.

I think you could eventually do something like `make CC=x86_64-w64-mingw32-gcc AR=x86_64-w64-mingw32-ar UNAME=MINGW` and it might work.


Inkább az assetEquals-t használd.


Just want to make sure there's no way for ApplyConfChange to fail. I seem to recall you mentioning some case where it might not succeed (a replica already exists on add, perhaps?).


I don't think this block is about excluding alternate means of responding to a challenge, it's about using the information we've got to see if a matching token works as expected.

I hadn't thought about whoami giving back a scope qualified username.  I'll have to talk to @liggitt about the long term vision there.  It's a little weird to have project admins aware of provider identity scopes when granting permissions.


you should use os.path.splitext() to get the extension and compare against that

also does this protect against case sensitivity? we may want to consider .lower() before comparison


getrachtest?


Yes, it seems to work okay: https://github.com/sjkelly/mraa.jl/blob/cb52eb2be4fdb8c80406a2a743c1e96018809368/src/enums.jl#L11-L13 It was just kind of annoying because I was copying from C, then I had to track down the repeat values once julia errored.

Optionally we could just print the first occurance, or all matching.
e.g.
```
julia> @enum Foo A=1 B=1 C=1

julia> A
A::Foo

julia> Foo(1)
A::Foo = B::Foos =  C::Foos
```


Should place that as a note below the example...


Sehr gut. Vielen Dank. :+1:


ditto


Should we pass in `user` here instead of `null`? It doesn't look like we're using the `expectedUser` anywhere


same with line 762 (what's up with the "MP4" check?)


Are you sure that's the right license?


typo


Whoami should return the simplest possible representation of user info, which right now is UserInfo.  But there's no guarantee that matches the authentication user name so we can't rely on that.  We would have to store the previous whoami and compare.

> On Mar 3, 2015, at 1:37 PM, David Eads <notifications@github.com> wrote:
>
> I don't think this block is about excluding alternate means of responding to a challenge, it's about using the information we've got to see if a matching token works as expected.
>
> I hadn't thought about whoami giving back a scope qualified username. I'll have to talk to @liggitt about the long term vision there. It's a little weird to have project admins aware of provider identity scopes when granting permissions.
>
> —
> Reply to this email directly or view it on GitHub.
>


Disregard. That commit is broken.


Damn link not working...


Should we also try to launch with an empty key or secret to ensure that doesn't work either?


ApplyConfChange will never fail. ProposeConfChange may drop the proposal on the floor if there is an existing proposal pending, but once we get to this point ApplyConfChange will succeed.


Это старый способ восстановления пакетов. Есть [новый](https://docs.nuget.org/consume/package-restore), с коротым меньше проблем.


not aim, it's mandantory to get code into coala. If code is untestable, one could add `# pragma: no cover` at the branch. (e.g. at the else if the else branch cannot be reproduced)

One should aim at 100% branch coverage.


Ez a regexp megeszik mindent a ".*" miatt. String.contains("3") célravezetőbb lett volna


Maybe just better to set r.Desc.Replicas = change.UpdatedDesc.Replicas


Why not just `size == fi.read(...)`?


Same here.


FTR, that should be a simple `static_assert`.


it actually checks if a **is** the `True` object, not something that is considered equal. (At least I think it does.)


`IAiProvider`?


This comment is possibly misleading wrt force uploading vs. force processing.


Ditto.


Yeahh, you're right in your comments, when I can I will make a new commit with those details solved.


Where do you add `#pragma: no cover` in the branch?


Nit: Two spaces before `//` please. :)


1540218-SlinkyGroundHX (mmatts) - x86_64-MacOS-10.9-clang: Tests Failed

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


e.g. `else  # pragma: no cover`


You probably want to round instead of truncating.
(Or perhaps multiply by 10 or 100?)


or `if something is True:  # pragma: no cover`


1540218-SlinkyGroundHX (mmatts) - i386-Windows-7-VisualStudio-12: Tests Failed

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


yayyy!


Можно утащить в конструктор


read the doc, seems so, will rechange that to "is" again.


I think this is sligtly broken because we may well have a buffer that **isn't** null terminated and in that case the strlen() will read beyond it. We can fix it by using strnlen() instead. Is that portable? Otherwise we can implement it ourselves.


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/915
failure: http://build.servo.org/builders/mac2/builds/1130


In your test file? or where?


strnlen() is IEEE Std 1003.1-2008 (``POSIX.1''), I guess that will do?


This factoring was easier for me than rewriting Semaphore to use a Condition.


saw approval from SimonSapin
at https://github.com/servo/servo/commit/1c4b0dda5517627e81fe9fb2df35a7c76281b31c


merging servo/servo/background-size = 1c4b0dda into auto


servo/servo/background-size = 1c4b0dda merged ok, testing candidate = e1a50c77


It would be nice to have something a bit stricter than `func` for this case.

Can you test whether `React.PropTypes.instanceOf(ReactCompositeComponent)` also works here?

Also, can you update your commit message with more details on how you tested this?


ah no, in the code which you are not able to test. Better example:

```python
# REASON
if sys.version_info < (3, 4):  # pragma: no cover
    pass  # Special treatment for older python versions
else:  # This code get's executed during normal test
    pass
```

It's important to also include a reason.


This code is much like in Condition.wait, but factoring it leads to *more* complexity, not less.


It really isn't, somehow I managed to apply the ASM license by mistake.


Yes, that's right, thanks.  I'll update the worker version in a bit.


We're exploring if there is a good way to generate "unique" ids without having to read to check if it already exists and lock them manually. UUIDs are nice but they are 16 bytes long... If we figure out a way we can remove both `id_name_lock` and `max_id`.


Test


Is "None" a magic word in msbuild collections?  I don't see anything that would purge the value if None were actually specified.


i fixed this a few min ago


We're exploring if there is a good way to generate "unique" ids without having to read to check if it already exists and lock them manually. UUIDs are nice but they are 16 bytes long... If we figure out a way we can remove both id_name_lock and max_id.


This will break a lot of libraries like Carbon that lets you pass both a DateTimeZone object and a string.

```php
$time = Carbon:: createFromFormat('Y-m-d H:i:s', '2015-03-03 12:00:00', 'Europe/Paris');

$time = Carbon:: createFromFormat('Y-m-d H:i:s', '2015-03-03 12:00:00', new DateTimeZone('Europe/Paris'));
```

The proposed "fix" above using `version_compare` won't solve this.


Ah good to know, will add.


This will need to change to the following:

relabel_dirs_pattern($1, device_t, { device_t device_node })
relabel_lnk_files_pattern($1, device_t, { device_t device_node })

to squash the following relabelto denials, which were consistently reproduced without the interface change:

type=AVC msg=audit(1424357208.664:46): avc:  denied  { relabelto } for  pid=1 comm="systemd" name="/" dev="devtmpfs" ino=3 scontext=system_u:system_r:init_t:s0 tcontext=system_u:object_r:device_t:s0 tclass=dir

type=AVC msg=audit(1424357208.665:58): avc:  denied  { relabelto } for  pid=1 comm="systemd" name="root" dev="devtmpfs" ino=7469 scontext=system_u:system_r:init_t:s0 tcontext=system_u:object_r:device_t:s0 tclass=lnk_file

type=AVC msg=audit(1424357183.585:46): avc:  denied  { relabelto } for  pid=1 comm="systemd" name="/" dev="devtmpfs" ino=3 scontext=system_u:system_r:init_t:s0 tcontext=system_u:object_r:device_t:s0 tclass=dir

type=AVC msg=audit(1424357183.586:58): avc:  denied  { relabelto } for  pid=1 comm="systemd" name="root" dev="devtmpfs" ino=7447 scontext=system_u:system_r:init_t:s0 tcontext=system_u:object_r:device_t:s0 tclass=lnk_file

The rest of the denials are relabelfrom, and should be handled as the interface is currently implemented. Thanks for catching this, @doverride.


Is there any way to also .gitignore this file by default? I'm assuming people will be using this to store sensitive credentials?


Test d'ajout Carl


Ne das sieht aber gut aus


> 1. There's one chdir() call in some Windows-specific code - not sure if it's needed but won't be used in the osx code anyway.

IIUC this one was "needed" not to lock the directory in which Geany was initially started (e.g. from which you opened the first file), as you know Windows doesn't allow suppressing directory in use.

> 2. But there are 2 rather wild-looking chdir() calls in vte_get_working_directory() - I think the code is supposed to just find the shell's working directory but not change Geany's working directory (probably caused by copy/pasting without thinking what it causes in Geany).

I was about to remove those, but I then wanted to check if our code was OK for uncommon OSes and import [changes from xfce4-terminal](http://git.xfce.org/apps/xfce4-terminal/tree/terminal/terminal-screen.c#n1880), but I then stumbled upon [this link](http://unix.stackexchange.com/a/94365) trying to find a better way of getting the CWD (as e.g. */compat/linux/proc/%d/cwd* doesn't exist in the [FreeBSD 10.1 VM images](https://www.freebsd.org/where.html)).  If I believe this guy, reading the link doesn't work on Solaris, and I'm way too lazy to try and find/install a Solaris installation to check.

So, as they should actually be harmless (it chdirs back) and only used on non-Linux, I'd rather keep them until someone really tries and check a change to these work everywhere.


Ouain ouain ouain


@aerych - This is iOS 8 only and is currently crashing in production.


No longer needs the Stream parameter


looks like a colon slipped in here :smile:


I submitted a pull request for this, although it seems trivial so you may as well do it manually :)


There are two Find Friends now


J'vais finir par le trouver


rename this so that it doesn't shadow "SPEC"


Yeah, but show[Raw] is really raw, like `Term.Name("foo")`


First "Find Friends"


Tests?


They're just shortcuts.


Yeah, but that's benign (as long as you don't consider random unnecessary code lying around to be harmful...)


And second "Find Friends"


I think I would go with full and nothing for the other one.


This should return `const std::string&`  --- this is both for efficiency (no copies) and correctness (elsewhere in the code you use text().begin() and text().end() and unless text() returns a reference the begin() and end() won't be from the same buffer!


If you are going to make query_fields a set, why not make it a set of `fields` and .add() the `properties` one by one?


(sp) description


so many changes @roxygen2


@aerych  here too


Why would you do this over and over for every field in fields?


They should be using environment variables for emotive information. We need to commit the deploy.js config file. Think of ember-cli-deploy in a continuous integration scenario. The CI server will be deploying. Therefore the config needs to be present.

If you're storing credentials in a config file, there are better ways.


Sorry, I've missed a warning here now that W4 warnings are enabled, I will fix it in next batch.


nice!


Have you ever thought about using Dplyr along with magrittr using mutate for readability


Csinálhattál volna egy paraméter nélküli konstruktort, mely meghívja ezt, s átadja a "," delimetert.


This code seems to be an exact duplicate of the code above.


remove this line


Lache pas la patate !


Unnötige Leerzeile


@philschatz Note that practice-problems will have multiple questions in it. All practice problems should be encapsulated under the same label Practice Problems.
@kathi-fletcher FYI: in the word doc, you commented there should be Practice Problem and Practice ProblemS. This is not the case. There is only one label Practice ProblemS in the design template. So this is correct. (There are Worked Example and Worked Examples, so you may be thinking of that.)


remove line seven and eight


Inkább konstruktor-ba való a delimeter megadása, vagy ami még jó, ha van rá setter metódus.


Shouldn't this be CredentialSupport.UNSUPPORTED?


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (629 of 629 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-3.96%25-green.svg)


Could you add a comment explaining the check here?


bitte wieder einkommentieren. ;)


*Less hard coded! :)


Typo: Should be 'Method'


Ah yes. I've just pushed an update.


Algo extraño ocurrió y volví a hacer commit de lo mismo por error.


Sounds like an ifdef (distro_redhat or distro_rhel7) should be sufficient here.


I'm not a fan of this change and I see having to pass in the raw value as a code smell. Maybe we just need to move these elements?

```

                                                 +--------------+
                      modifies data              |presentation  |
             +-----------------------------+     |              |
             |                             |     |              |
             |                             |     |              |
             |                             |     |              |
             |                             |     |              |
             |                             |     |              |
         +---v----+    +---------+     +---++--+ |  +--------+  |
         | data   |    |filter   |     |behavior |  |formatter  |
         |        +---->logic    +----->       +---->        |  |
         +--------+    +----^----+     +-------+ |  +---+----+  |
                            |                    |      |       |
                            |                    |      |       |
                            |                    |  +---v----+  |
                            |                    |  |search  |  |
                            +-----------------------+        |  |
                                                 |  +--------+  |
                                                 |              |
                                                 +--------------+
```


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


82420068_GridViewMultiEdit (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Can you add "Added support for Hikari JDBC Driver" here? I know its not part of your commit, but it is technically part of 1.7.11


Does that actually change the return value, or just provide a compile time check that you're always using a const ref?


This is a hard one, I'll try.


@philschatz We decided to make these ost-example instead of ost-reading to be more generic.


Ügyes, hogy a végén levágod a felesleges két karakter. :)

Vannak elegánsabb megoldások is:
 * Van az apache commons-lang nevű lib, amiven van egy StringUtils osztály, csomó hasznos metódussal, az egyik a join, itt megnézheted (lehet még később jól jön): http://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html#join(java.lang.Iterable,%20java.lang.String)
 * Másik lehetőség, hogy olyan siklust csinálsz, mely csak akkor fűz a string-ed végére vesszőt, ha van még hátralevő elem a listában.


Yes it changes the return value from a copy of the string to a reference to
the string.

On Tue, Mar 3, 2015 at 2:00 PM, Hadley Wickham <notifications@github.com>
wrote:

> Does that actually change the return value, or just provide a compile time
> check that you're always using a const ref?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hadley/fastread/commit/3738d54b9125aad703575ff5e6b64e276686e5ab#commitcomment-10013869>
> .
>


Miért kapod el és dobod tovább? Engedd csak nyugodtan ki.


Any performance improvement should start with a profiling session to check if it's even worth to focus on optimizing this or that part of the code. I currently have no idea about the impact of i_visibleNow on the overall CPU time


Valójából a második add() már meg se hívódik.


Yeah, i rewrote that part anyway, no need to zend_string_copy as it's the same pointer. Thanks.


helyes, megtaláltad ez az eszközt! :)


Maybe. Should it still pass the full descriptor in the trigger? I'd rather pare it down to just the parts that are used, but split and merge pass the full descriptor and don't use the whole thing.


Well, that's weird, I don't see why this change would produce any differences. Do you know which one of the three lines makes a difference ?


Maybe add some comments why this change makes any bit of difference


1. Лучше под это создай отдельный репозиторий, не продолжай писать в частотный словарь.
2. Программирование почти все про правильное мышление, как можно правильно мыслить, когда у тебя в голове категории вроде scts? Надо всегда и везде использовать читаемые имена. Чтобы и другой человек мог понять, о чем будет этот проект.
3. Никогда не храни старый код в комментариях, для этого у тебя есть ветки, система контроля версий и тд.
4. Попробуй передавать пароль в открытом виде через sshpass http://sourceforge.net/projects/sshpass/


@philschatz we decided to replace ost-interactive with virtual-physics everywhere


Line 762 is redundant. You just need this
```
	if (from_t != e_b64_raw)
		from_len = strnlen((char*)from, from_len);
```


No worries. I missed the original comment by @sklum and had a bit of a wtf
moment b/c I though Bin errors were a thing of the past :)

Thank you again for all the effort on this refactor. OpenBR is a lot more
dev friendly IMO.

On Tue, Mar 3, 2015 at 1:10 PM, JordanCheney <notifications@github.com>
wrote:

> @bklare <https://github.com/bklare> sorry, I thought this was a quick
> solution for Scott's issue I didn't realize you were using histbin. From
> this point forward, I will consider the reorganization changes as stable
> and submit PRs for any changes I make.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/biometrics/openbr/commit/7e1cff9ce5482b358bcfed89d02b6d011bd17f33#commitcomment-10012905>
> .
>


Does this make a noticable difference?


Instead of talking about triggering the resize event manually, simply document that `eqjs.refreshNodes()` can be used to update the listing of nodes that is use by `eqjs.query()` with all of the nodes currently in the DOM.


Add to build so that available for AWI to deploy.


Also h5py is required too?


Just pass the replicas. Split and merge need start/end keys + the replicas,
so the whole thing is more appropriate.

On Tue, Mar 3, 2015 at 2:04 PM, Ben Darnell <notifications@github.com>
wrote:

> Maybe. Should it still pass the full descriptor in the trigger? I'd rather
> pare it down to just the parts that are used, but split and merge pass the
> full descriptor and don't use the whole thing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bdarnell/cockroach/commit/9b3578f863c2b13b4f5d2da80f16e0df3d3faf68#commitcomment-10013938>
> .
>


Használj majd kódformázást (tudod ALT+SHIFT+F), tagoltabb lesz a kód, így könnyebben volvasható.


Для чего это?


Changed in 049828a   but this still may not be 100%.

Note, in the case we had, it would work, since length was 50 (or something), while length of data was 44..So the null would 'save' us from walking off the end.  BUT if like you say, there was no null, and the length was too long, nothing saves us, not strlen, nor strnlen.  We walk of end of buffer.  Nothing I can do about that, BUT this above change does correct almost all code that will be thrown at this function.

Originally, I had code that did the strlen, BUT I undid it, due to passing in a string, and only wanting the first X bytes of it, instead of the whole ASCIIZ length.  This code I have done now, still preserves that ability (to grab the first 8 bytes from a 40 byte hex string, i.e. 4 raw bytes, and then later grab the last 32 bytes, which are the checksum).  That ability is still there.  Since I only look for the passed in from_len being LONGER than the from string, it will only shorten it if there is not enough data.

Also, this code will ONLY work on the ASCII data types. raw memory will never be checked, since we have on portable way to know if we can use that much buffer or not.  The caller has to get it right.


Okay I don't quite understand that. How do you mark for example a function in your added component that it shouldn't be included in coverage analysis? So you have
```python
class YourComponent:
    def func1(self):
         # Does something untestable.
    def func2(self):
        # Does something that is testable.

class ComponentTest(unittest.Test):
    def test_func2(self):
        # Test whatever...
```


Remember to store JOGL library as JOGL-JARS in the lib path


I forgot what we decided here. Shouldn't we do a bing seach in cases where helplink is not set (similar to what we do in preview pane and error list)?...


```python
class YourComponent:
    def func1(self):  # pragma: no cover
         # Does something untestable.
    def func2(self):
        # Does something that is testable.

class ComponentTest(unittest.Test):
    def test_func2(self):
        # Test whatever...
```


make these module level


in the spirit of not having 'magic numbers'


Remove the `separating each, with commas`


Don't forget to document why this folder is added as a resource dir.


Should link these out to the github pages


Yes, added.


Um?


Short description?


should be
```
```hbs
<a href="{{url}}">
```
```


May I delete this branch?


AKA: Issue01


Itt használhattad volna a this(DEFAULT_DELIMITER); is


I see no case where we will fail now, except if someone passes a from_len that is larger than the buffer - but that would be a bug in the caller.


et si il s'est passé ≥10ms ?


Remek, hogy megtaláltad!


I can't say that I've ever used images from repeaters in this context, but
think I know what the issue is. I just pushed a minor change in the last
commit that I think will fix it, can you confirm? (I don't have any good
test cases)

Regarding the "select another image" that's a good point. Though the term
"another" can mean "different" or "additional", both are accepted
definitions. Not sure I want to force re-translations in all the languages,
unless/until we have some capability to select multiple images at once (at
which point I agree the term would definitely be too ambiguous).

On Tue, Mar 3, 2015 at 12:59 PM, adrianbj <notifications@github.com> wrote:

> Hey Ryan,
>
> This is very cool - not being able to do this has definitely been an
> initial source of confusion for some clients.
>
> The only weird thing I am seeing is that once you upload a new image, the
> grid of thumbnails no longer includes images from repeater fields within
> the page. They are shown when you first click the CkEditor image button,
> but are gone once a new image has been uploaded.
>
> Also, just noticed the wording of the "Select Another Image" button. This
> sounds to me like you can select more than one image - "Another" suggests
> one more. Perhaps "Select Different Image" or even just "Select Other
> Image".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ryancramerdesign/ProcessWire/commit/c2d42675f79a1f0aae3702a6a867877e691e133c#commitcomment-10012627>
> .
>


What is this class doing?


Both should work actually...


@dkayiwa I've confirmed that this breaks the patient dashboard for a dead patient.  I'm going to revert.


You can use quotes to your advantage though, for example:

```
<a href="/something/{{path}}/{{basename}}.{{ext}}"></a>
```


Ah nice, feels weird though, without the `"`, @rwjblue


cd .vim
git submodule init
git submodule update


Did not understand where the optimization was, but maybe not so important.


Szép hogy így oldottad meg!

Lehet, hogy érdekel van az apache commons-lang nevű lib, amiben van egy StringUtils osztály, csomó hasznos metódussal, az egyik a join, itt megnézheted (lehet még később jól jön): http://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html#join(java.lang.Iterable,%20java.lang.String) A dependency: http://mvnrepository.com/artifact/org.apache.commons/commons-lang3


Totally agree


Hi Eric, Bradley from Hannon Hill here. Any chance you could clean up this README a bit? Not quite sure but what happened but I'm having trouble viewing the rendered version.


Where does this data format come from?


I think it is all there.


Oh thanks for catching it. :)

On Tue, Mar 3, 2015 at 10:15 PM, Mark Goodrich <notifications@github.com>
wrote:

> @dkayiwa <https://github.com/dkayiwa> I've confirmed that this breaks the
> patient dashboard for a dead patient. I'm going to revert.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-coreapps/commit/0924e41cc9d99995d9afa08f25e02108b323bc6f#commitcomment-10014142>
> .
>

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


Thanks for the comment but this has been already fixed in a newer commit: https://github.com/tus/tus-resumable-upload-protocol/commit/134f99a4c21b300fea86d7ba32dd02ee8672b51b#diff-7327496955357b9792bab8720dc507d6R202


Dont use the cursor. It's going away in the next version of the autocomplete API. Use `position` which is the buffer position and is passed into the `requestHandler` options.


"sera ajouter" ? sérieusement ? (Electronique.tex)


this is shit


PRs welcome, autocomplete-css and autocomplete-html use the cursor too


this is tchernyovyick :) WIP still


J'espère ne pas m'être planté ceci étant ;)

--
@mobile

> Le 3 mars 2015 à 18:56, Romain Bouqueau <notifications@github.com> a écrit :
>
> excellent :)
>
> —
> Reply to this email directly or view it on GitHub.
>


Why don't you just change the path to '//developers.eve....'. This way the browser chooses the protocol automatically.


@philschatz

DECISION: These elements should be added to the Tagging Legend and should be discarded from the reading.

Section Key Terms
Section Summary
Key equations
Answer key

Key Equations

You already added key-equations in a prev commit, but we should ahve the others as well.


@danielballan says: "Define specs"


82420068_GridViewMultiEdit (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

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


Yeah, doesn't seem bad that way.

On Tue, Mar 3, 2015 at 8:54 PM, Underground78 <notifications@github.com>
wrote:

> I think I would go with full and nothing for the other one.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Underground78/mpc-hc/commit/d9139904974d8544391188ebaa902458c424525f#commitcomment-10013724>
> .
>


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


all tests pass:
success: http://build.servo.org/builders/linux1/builds/916
success: http://build.servo.org/builders/linux2/builds/914
success: http://build.servo.org/builders/android/builds/952
success: http://build.servo.org/builders/mac1/builds/1131
success: http://build.servo.org/builders/mac2/builds/1131
success: http://build.servo.org/builders/gonk/builds/102


remove repo


I did make some helper methods like `Location.toBlockPos()`, but a lot of those methods had to come straight from Minecraft code and I don't know that it can be improved that much. Besides, those ugly code segments were used to make utility methods, so we should at least have those from now on.


Because of file:///, all the assets are going to be remote but it's handy to be able to test the file locally


There were other issues with a previous strategy. The only difference should be that instead of on `$em->persist($entity);` it will update created at only on `$em->flush();` for all newly persisted entities. The tests shown that behavior has not changed as far as tests are covering. I have also updated tests in order to prevent **sleeps** which were used before.. please note if you find something differently working


@zcbenz i made a few assumptions here, that request data will be a node buffer and sending the response with the data extracted from it. Is that correct ?


@cgardens Hey man, is this fully functional?


Kikerüljük az elválasztót :)


I for one demand to know what this class is doing!


@zcbenz based on the assumption that data will be a node buffer , is this the right way to extract it from mate::dictionary ?


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-MacOS-10.9-clang: Tests Failed

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


fast-forwarding master to auto = e1a50c77


Inkább dobd a kivételt!


Does a slice really need to have a destructor? This doesn't seem conventional.


  //1. Create a webfile
  var webFile = new WebFile(startPath);

  //2. Create a Compiler Object
  var compiler = new Compiler();

 //3. Pass the webfile to the compiler
  var compiledFile = compiler.Compile(webFile);

  //Use the result to directly send it to client ....

 //or you can save it
 compiler.Save(@"C:\CompiledIndex.html");


Erre van jobb módszer is. JUnit oldalán megtalálod. Órán szóljál, megmutatjuuk.


Thank you very much!


If the bosun command is the only thing to use this package, it should be under the bosun directory. If scollector also uses it, it can stay here.


In fact, I tested with TextField from Material-UI, rendering works fine but updating text inside the text field will trigger an error. The reason is that TextField seems to have its own ref property and Typeahead needs to find the entry reference. When I replaced this.refs.entry with event.target in your code, everything worked fine but I don't know if you are okay with that, hence as I said the pull request is still incomplete :)


I will change this, and also implement our own strnlen() unless system has one. We already used it in formats.c.


comentario de la linea para seguir haciendo cosas


Meg is találtad, szuper! De akkor miért hagytad meg azt a másikat? (A teszteset jó, csak a módszer kerülendő)


Ooof.


@openstaxalina ost-assessed-feature = element with instructions, text (and possibly video, but doesn't have to have a video), and a grasp check. Examples =
fun-in-physics
work-in-physics
boundless-physics
link-to-physics


What about additional libraries that libpcap requires, if any?  (It may not require them when linked dynamically, as the libpcap shared library will be linked with those libraries, but it may require them when linked statically.)


> the synchronisation step sounds like a pain

indeed. I created a `sync-travis` plugin that I'll publish, but it doesn't really solve the problem. For me it takes 1-2 minutes for everything to sync before I can enable a new repo. Seems like Travis CI should allow you update with one project, instead of having to sync all of them.

> I would love to add this as a publish step

that's great!


this is how I tested
```javascript
var Mui = require('material-ui');
<Typeahead CustomComponent={ Mui.TextField } />
```


I never did come up with a solid repro case, I just force `immediate` to `false`:
https://github.com/yahoo/express-busboy/blob/master/index.js#L21


Por fin repair #3


close #3


Change this line
String inputLine=input.readLine();
to readbyte


Presumably you mean "libpcap.pc.in" here, to ensure that it's included in the release tarball.  We don't want to put the *generated* file in the tarball.


remove less than “<“ character from the url


Thanks for the link. I read through it but I'm not sure what it answers. I
did find this:

http://www.smashingmagazine.com/2014/03/04/introduction-to-custom-elements/

So I am guessing this is what I am doing with <stars..... />

Still unsure about caching and passing the csrf token and list of id's to
possibly avoid the GET completely.

I also wanted to ask you the best way to get the id's from localStorage so
an anonymous user can also have the starred page.

On Tue, Mar 3, 2015 at 11:04 AM, Peter Bengtsson <notifications@github.com>
wrote:

> https://github.com/bendc/frontend-guidelines#semantics
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bugzPDX/airmozilla/commit/01e9790ce2ed6aa05bd5db29403442d83a09bdb7#commitcomment-10013950>
> .
>


I really don't like doing global initialization here. Without some non-global means of configuring HTTPS (which I don't think is possible with our current infrastructure) we should just require users to configure this externally.


tagline for “Trust” vital should be “ Our security and privacy are respected” (there was a discrepancy between tabs in the spreadsheet - now updated)


Phil, can you add an example of when this is used?


“Openness” vital tagline should be “ Fair and transparent rules preserve our independence"


None is a fake category. Since there is no way of providing empty property as it will be replaced with the InnerLoop we need some fake category. Since no one is ever gonna add category with the name "None" then it is fine to assume that it will run the tests with all categories.


remove locale from URL


I don't think we should provide this functionality. It assumes that the hostnames are configured properly and flat-out will not work on `localhost` as written. It requires enough decisions about behavior that it should be left up to external users.


Done in e7fd3cc


I'm curious why you did the `!__MonoCS__` workaround here?


Don't link to a personal gist in package documentation. If there's additional documentation that's needed to use this, include it inline in the doc comment.


82420068_GridViewMultiEdit (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Unfortunately that seems to be worse - now I can't even select images from within the repeater - so if I go to insert an image into a body field in the repeater, the only images that are available are from the main page's images fields. The images in the image field inside the repeater are now not available at all,


@openstaxalina yes -- snap labs will be discarded from the interactive reading. tutor only problems will also be discarded. see below: ost-exercise-block


I think this line is the only line that may be conflict with upstream code base of gatk. In the line I want to suggest moving the links for binary distribution to the final `target` directory like the way finale Queue/Gatk jars are being linked.


nice easy fix!


How do you encode these types?
I just added matrix and vector types and added a field for float array encoding.
See 9e73210.


I agree. However, there is absolutely no documentation about this whatsoever. I spent a whole week figuring out how to do this, so perhaps the Dart website should publish a tutorial for it.


Элемент бэмовского блока не может существовать вне этого блока. У тебя создан элемент main-nav__list, а самого блока с классом main-nav нет.
Пропиши лучше тегу nav класс main-nav, это будет блок, внутри него уже положить ссылку на открытие/закрытие меню main-nav__toggle-link, например, и список с пунктами меню main-nav__list
 думаю вообще не потребуется, так ссылка которая открывает/закрывает меню может менять свой внешний вид в зависимости от состояния: гамбургер - меню закрыто, крестик - меню открыто


Other dependencies needed in Ubuntu are glade, glade-gnome and glade-gtk2


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


It's a question, should have a '?'.


Spelling mistake !!


In xunit.xbuild.sln, I removed the TD.NET runner and tests, so this would cause a compilation error when using xbuild.


Originally it was targeting some strangely named vendor files. This was just carry over which has been fixed. Should be good to go.


Переключатель слайдов в отзывах и тарифах выглядит совершенно одинаково, возможно, есть смысл его сделать отдельны блоком, независимым от того, где он находится. Можно задать что-нибудь типа "slider-pagination"


Ezt az if-et kivihetted volna egy metódusba, mondjuk valami validate-be, mert azt csinálja, hogy leellenőrzi a bejövő paramétert, ha nem jó, akkor dob egy kivételt.
Sőt a mach paraméterét azt a gyönyöriséget is kivihetted volna egy metódusba.

Csak annyi hogy olvasmányossá válik tőle kód.

Pl:

validateInput(numbers);
...
private void validateInput(String numbers) {
    if (!numbers.matches(getProperInputRegexp())) {
        throw new IllegalArgumentException("Bad character(s) in input string: " + numbers);
    }
}

private String getProperInputRegexp() {
    return "-?[0-9]+((" + Pattern.quote(separator) + ")-?[0-9]+)*(" + Pattern.quote(separator) + ")?";
}

Mondjuk ez utóbbit is még lehete csinosítani picit (kétszer van a Pattern.quote(separator)).

Remélem azért látszik mi volt a szándékom ezzel.


I don't have this decision recorded in my notes. @philschatz?


test


Fine with me :+1:


Could this have broken the MassDOT logo size?


LGTM


Здесь тоже элемента блока есть (contacts__header, contacts__title, contacts__adress), а самого блока нет (contacts)


nice easy fix


Ehhez önmagában is csinálhattál volna tesztet.


So is this necessary because it is possible for pValue to occur multiple times among the items of tCurrentPreferenceValue?


:+1: **SUPERLIKE** :+1:


@pl - I'd like to get your input on the callback for the new Event Buffer stuff.

There was a consensus that domain specific objects (abstractions) should be exposed that made it easy to access the responses from the API. This mirrors that we don't ask people to call `post` in order to trigger an event.

Node/JS is an interesting one since it's so easy to parse a JSON response. But maybe we can do a bit to help the developer?

1. What do you think of the suggestion above?
2. Do you think the request and response should still be exposed? If so, on the result object or as additional callback parameters?


Merci ;)


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - 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)


I wasn't able to test this because I'm on a spare laptop at the hospital, but it should work. If you get a chance to test it and it does work let me know and I'll merge it down


was divclass


Do we need to introduce a whole new lookup function? We're only using it once here. Can't we just use `not . null` on the existing lookup function that returns the list of packages by name.


Why not leave them in there? Depending on `__MonoCS__`  is considered a bad practice.


:+1:   I like this change.


QwtUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Majdnem :)
Igen kellenek azon maven varázslatok és a shippable.yml és a .gitignor-ba fel kell venni a következőt is új sorba: shippable


Actually, this file IS needed to turn the Harold directory into a package...


just noticed we could chain/group some of the z.X.on for speed


It is the suggested way to do it as per Dart tutorial: https://www.dartlang.org/docs/tutorials/httpserver/#using-https
Making the call is required and making the user do it himself will confuse most of the users.

Those that know about the call and need a more advanced setup, will recognize the parameters and can do it manually if preferred.

In the trade-off between usability and support for edge cases, it should definitely be included in my opinion since it does not rule out any edge cases whatsoever.


Check license


should also be "utm_content=6"


1: M111 unrecognized component name: BibEdit


Köszi, hogy felhívtad a figyelmem ezekre, szerintem ez a legnehezebb része, hogy mennyire bontsak szét valamit.


Interesting, I didn't know about scholdoc.

I guess we should do the same with the messages in the minibuffer...


Thanks for all the comments Jasper. I've addressed most of them and published the tutorial at the [School of Haskell](https://www.fpcomplete.com/user/meiersi/the-service-pattern).


This is used by unit tests. I think we'll be okay. ;)


I normally avoid posting these, should I post them? They come up every time i do anything with unity.


Thanks.  Updated to 8 and 16k respectively.


А здесь в отличие от пинка ссылки открытия и закрытия разные и делать их при помощи одного класса не получится, лучше разделить отдельно на закрытие и на открытие
Опять же есть класс элемента main-nav__list но отсутствует класс блока main_nav


do you have access/"owner"-permissions for https://jira.grails.org/browse/GPDATABASEMIGRATION ?

while i do not mind at all checking your fork from time to time for activity, it would be good to have some sort of issue to be able to track progress regarding grails-3 support. and of course to avoid somebody else going down the same route seperately instead of contributing.

what do you think? and ofc **thanks** again for your work so far! zyro


@steve8708 do you remember why you added `_.compact` here? I believe `repeatExp` just a string at this point but want to make sure I didn't miss anything before removing it.


This new documentation section doesn't show up here: http://gpkit.readthedocs.org/en/latest/advancedcommands.html, presumably because it is part of next release. Just want to make sure -- is that expected?


Looks good


LGTM (modulo cosmetic comment)


il y a plein d'endroit ou j'ai laissé la virgule, par facilité, je peux aussi les enlever si tu préfères


Which is why this isn't necessary. There is no need, that I see anyway, to make this a package.


This still feels a bit clunky with the control flow and having the `returnInitFlags` pulled out.

How about something like:

    chooseAgain <- if isPkgRegistered pkgName'
                     then promptYesNo promptOtherNameMsg (Just True)
                     else False

    if chooseAgain
      then getPackageName sourcePkgDb flags
      else return flags { packageName = maybeToFlag pkgName' }


Már csak a hibaüzenetet kéne megnézni, hogy jó-e. Órán szóljál, s megmutatjuk.


This will totally work well


Van jobb módszer.


Just a comment onn a request


I've got it set up. I'll let you know if I get updated.


TeamCity Akka.NET :: Akka.NET PR Build [Build 92](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=144&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


Hi


You're not alone!


Для блоков на синем фоне или на сером фоне точно потребуется модификатор, чтобы прописать ему другой цвет фона


89224666-FixNANProblems-Issue4557 (rraustad) - i386-Windows-7-VisualStudio-12: Tests Failed

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


bla-bla


Hmm, I might need to just setup a repeater here so I can have a test case.
Before your first message, I'd completely forgotten that one could even
select images from repeaters (and don't remember why exactly we even
support this). But I just pushed a update that might fix it? If not I'm
going to try and setup a test case here.

On Tue, Mar 3, 2015 at 2:34 PM, adrianbj <notifications@github.com> wrote:

> Unfortunately that seems to be worse - now I can't even select images from
> within the repeater - so if I go to insert an image into a body field in
> the repeater, the only images that are available are from the main page's
> images fields. The images in the image field inside the repeater are now
> not available at all,
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ryancramerdesign/ProcessWire/commit/c2d42675f79a1f0aae3702a6a867877e691e133c#commitcomment-10014479>
> .
>


Yep, seeeeya. :+1: Foundation has been totally botching anything that uses Compass... if I upgrade to the latest Compass, Foundation breaks. Without the latest version of Compass, Bourbon wouldn't work with it. And I still need to easily work on my old Foundation sites. So instead of switching between versions of Compass every time I switch projects, I shut that sucker off. Thanks a lot Foundation. Jeez.


@dkayiwa I believe this update broke the core apps build.  Tests are failing with the following error:

build	03-Mar-2015 14:36:38	Caused by: org.springframework.beans.factory.BeanDefinitionStoreException: Failed to read candidate component class: URL [jar:file:/home/bamboo-agent-2/.m2/repository/org/openmrs/web/openmrs-web/1.11.1-SNAPSHOT/openmrs-web-1.11.1-SNAPSHOT.jar!/org/openmrs/web/OpenmrsJspServlet.class]; nested exception is java.io.FileNotFoundException: class path resource


I suggest doing the easiest and most obvious thing first: iterate through the whole list of blocks looking for one that touches, where touch = shapes overlap. This is obviously right. You can make it more complicated/faster later if that matters in practice.


Calling `stop()` in the `m_reconnecting` callback will hang waiting on `m_start_completed_event` to get set.


Тип date для количества человек точно не подойдет и добавь классы элементам формы и лэйблам, без них проблематично будет так разметить


Nem unit tesztekről volt szó?!


Опять же нет родительского блока с классом social-buttons


develop (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


remoteObject.value = expression + " => Not supported";
     EvaluateResponse response = new EvaluateResponse();
     response.result = remoteObject;
     response.wasThrown = true


Is there any real benefit to having XMLWriterPreferences be global in the first place? This is ought to be in the configuration settings for the writer.

Also, I am not a big fan of  the way that Format is used as a places to stash parse results as well as a place to set options.


ONE


Модификаторы указывай через двойной дефис icon--htmlacademy icon--a


can't you ignore idiots instead of removing very useful and working well functionality?
in future they will enable plug-in and start complaining again.


konstruktorban van setterben lenne érdemes megadni az alapértelmezettől eltérő elválasztót.


Yeah, though we shouldn't spread the bad practice if it can be easily avoided IMO :) I can send a PR that adds the tdnet projects back if you agree.


The check is needed, s1disks being None would make the generator raise an exception. It doesn't really need to check for None though, and empty set will also result in the first one being picked.


public class Car
{
}


Patch looks ok to me.


I noticed this too. Though wondering why it did happen locally.
But for now, you can revert this as i continue to look into it.

On Tue, Mar 3, 2015 at 10:49 PM, Mark Goodrich <notifications@github.com>
wrote:

> @dkayiwa <https://github.com/dkayiwa> I believe this update broke the
> core apps build. Tests are failing with the following error:
>
> build 03-Mar-2015 14:36:38 Caused by:
> org.springframework.beans.factory.BeanDefinitionStoreException: Failed to
> read candidate component class: URL
> [jar:file:/home/bamboo-agent-2/.m2/repository/org/openmrs/web/openmrs-web/1.11.1-SNAPSHOT/openmrs-web-1.11.1-SNAPSHOT.jar!/org/openmrs/web/OpenmrsJspServlet.class];
> nested exception is java.io.FileNotFoundException: class path resource
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-distro-referenceapplication/commit/8dc02fcdf2e0281da413af23047feb44641cfa3c#commitcomment-10014717>
> .
>

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


Actually, how did you come up with 8k for net?  I would think that keeping it around the MTU (under 1500) would be better?


Dobunk kivételt. Szuper!
De elkapjuk és kiírjuk a szövegét... Akkor miért is dobtunk.

A kivételeknek az a céljuk, hogy hibát jelezzenek. Így most te a hibás esetet elnyelted. :(
Szóljál órán, segítünk, elmagyarázzuk.


Sure. Want to give it a go? :)


1540218-SlinkyGroundHX (mmatts) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


SPO-10 Lights are missing. Will add in on next pull request


Здесь думаю нет смысла делать ссылку отдельным блоком, сделай ее лучше элементом блока main-nav


Mintha kihagytad volna a teszt osztályodat.

Illetve látom a .gitignore nagyon hiányzik. Olvas el majd még egyszer a levelemet (a git-es előadás utánit)


we should log here and in the analogous case below.


Думаю здесь ты имел ввиду doWnload ;)


I’ve pulled it to https://github.com/mina86/urxvt-tabbedex, thanks.


А здесь лучше тоже исправить опечатку: вместо conrtrol -> control


I was actually surprised to see support for that also, but I was testing repeaters with one of my modules the other day and noticed that it was.

Anyway, your latest commit seems to fix all scenarios - only issue is the following warning:
```
Warning: Invalid argument supplied for foreach() in /wire/modules/Process/ProcessPageEditImageSelect/ProcessPageEditImageSelect.module on line 277
```


It's not usual to use typography to emphasize the key paragraph of a chapter or section. If one feels it's necessary to point out that something is important, usually that's done in the text itself, by saying something like, "The solution to our problem is..." or "The key idea is..." Or else it's the last paragraph, and everything builds up to it logically so it's clear that it's the important part. Anyway, usually we hope the reader figures it out on their own.

If you want to highlight something, rather than the whole paragraph I would suggest picking out a phrase -- to avoid emphasis fatigue. In this case, I think "counting the number of bounces in the direction of gravity" would qualify as the key phrase in the paragraph.


the part in brackets doesn't fit anymore


TeamCity Akka.NET :: Akka.NET PR Build [Build 92](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=144&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:09:16
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.PersistentActorSpec.PersistentActor_should_reply_to_the_original_sender_of_a_command_even_on_PersistAsync: <no details avaliable>

```


>  Is there any real benefit to having XMLWriterPreferences be global in the first place? This is ought to be in the configuration settings for the writer.

No!  It's a historical cockup/error.  It should really be changed. (In a backwards compatible way ideally).


Модификаторы элементов и блоков не могут ипспользоваться без основных классов элементов и  блоков. То есть при использовании модификатора у тега будет два класса class="price-table__cell price-table__cell--yes"


this should be :symbolId, not *symbolId or else bad things will happen


and it should be a good one. Also: Coverage is measured against python 3.4, so thats the reference!


Cool, I will revert for now


This commit creates a condition of index out of bounds for my instance.

last_file = db.list_tasks(limit=1, category="file", not_status=TASK_PENDING)[0].to_dict()["id"]


Might be simpler to change `index` to a regexp match, and do the split etc. in one go.
I tried to stick to the style of this file for local consistency.


likewise for anywhere that does this for something that isn't a string


de ez a string nem üres :)


It's release? "21 days ago"


I don't think this should be an official issues site. Either official repository should be mentioned when it is ready, or the link should link to the forum. Or it can be ommited completely.


Should that be in misc.[ch] and not in jumbo.[ch]  ??  This will likely not be something in core, and jumbo.[ch] is the warehouse for jumbo stuff, I thought


See commit message.


aye will do it shortly


this is the only include you need


all tests pass:
success: http://build.servo.org/builders/linux1/builds/903
success: http://build.servo.org/builders/linux2/builds/901
success: http://build.servo.org/builders/android/builds/939
success: http://build.servo.org/builders/mac1/builds/1118
success: http://build.servo.org/builders/mac2/builds/1118
success: http://build.servo.org/builders/gonk/builds/89


AdvancedAFN (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 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-3.67%25-green.svg)


Ah, I didn't disable the warning on GCC, just stopped it being an error. I think it's useful to keep as a warning. I actually don't know how to make a specific warning not an error under /WX on MSVC, I''ll have to investigate.


PrettyGridView (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2190 of 2204 tests passed)

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


fast-forwarding master to auto = 290b025f


I have got errors like #49 caused by this commit.(npm version is 2.6.1)
So, I have to use =5.0.0 currently.


No. I'm just making some tests to implement solutions into ongoing project. Why are you asking? By the way: is there any way to set some king of 'global darkness' and have light only were I want it to be?
When I was implementing 2d light engine some time ago, I've been filling each light map with some very dark grey colour ( almost black one ), and adding shadowed light texture onto. The effect was quite realistic then. Now everything there is bright even when there is one light.


iteration (kbenne) - i386-Windows-7-VisualStudio-12: OK (1620 of 1620 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)


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


saw approval from mbrubeck
at https://github.com/psdh/servo/commit/cb6c675b2e28e61237f2651ad0cff10e5aa137c5


merging psdh/servo/improveBinRun = cb6c675b into auto


psdh/servo/improveBinRun = cb6c675b merged ok, testing candidate = 801b9394


I fixed it for Linux.


微信6.1里分享没有反应，不管是分享好友还是朋友圈都没反应，android的全是，也有反应ios有问题的，这个有办法解决吗？
打开demo也是同样没有任何反应。


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


You can remove the theme block here.  You only need it if you intend for your plugin to theme the site.  Removing it prevents it from being included in the available themes drop down.


You will also want to unregister your nav item in the ```onUninstall``` call.  This ensures that items are tidied up.


test


SimpleCRG should use the ``default_clk_*`` values instead of duplicating this information.


You right


Your grammar is ambiguous. Drop by and I'll tell you a simple way to make the grammar (and the parser) respect precedence.


Another would be the reverse, -DLIBPREFIX and <set value="LIBPREFIX" value="lib" if="static" unless="LIBPREFIX" />, that sort of thing, so you can override it and force it to be a value you expect


AdvancedAFN (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1621 of 1621 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)


Or perhaps I should submit a pull request ;)


89435026-initialize-SysTotCoolCap (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1614 of 1614 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)


> maybe infinite is incorrect.

You mean the tag name is incorrect?


Told you I couldn't test it lol


well yes.  if you want it to be not dumb.